Update county validators #118
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ly set libraries based on verbosity
… for structured extraction
…text processing logs
…h _MofNCompleteColumn for improved display of completed tasks.
…rdinance extraction and county utilities
ppinchuk
added
enhancement
github-actions bot
pushed a commit
that referenced
this pull request
Apr 16, 2025
* Add new kwargs for prompt * Update name and jurisdiction prompts * Don't lean so heavily on zoning, since we look for other kinds of ordinances now * Add emphasis to prompt * Minor prompt clarification * Add clarifications to prompts * Fix density description * Add clarifier about density units * Increase location thresh * Fix docstring * Add total cost to meta * Add cost via tracker * Optional arg * Pass tracker to updater * Allow Utility and commercial "tech" to be treated similarly * Add geopandas as a dev req * `prop_line` -> `property line` * `struct` -> `structures` * Format scores * `rail` -> `railroads` * `trans` -> `transmission` * `density` -> `tower density` * `max size` -> `max project size` * `max` -> `maximum` * `min` -> `minimum` * Define `LLMConfig` interface and implement `OpenAIConfig` * namespace imports * Rename arguments * Deprecate old class * Update tutorial * Rename var * Rename function * Emphasize numerical * Clarification for numeric ords * Add log call for cache * Add ability to specify DL prog bar * Short pause to acknowledge PB completion * Add `TempFileCachePB` class * Use `TempFileCachePB` class * Update log message * New compass exception * Better control flow for tree evaluation * Catch new error type * Add `_setup_logging_levels` function * Use new `_setup_logging_levels` function * Refactor `_setup_cli_logging` to accept verbosity level and dynamically set libraries based on verbosity * Update verbose option to count and adjust logging setup in CLI * Clarify logging output message in async decision tree * Update logging levels and messages for clarity across parsing and downloading modules * Refactor logging setup in `_setup_logging` to only attach logging for COMPASS * Remove unnecessary clarification about error-level messages in README for structured extraction * Replace `logger.debug` with `logger.debug_to_file` for ordinance and text processing logs * `DEBUG` -> `DEBUG_TO_FILE` when running script * Logging call before progress bar starts * `_MofNCompleteColumn` now allows customizable text style for completed/total display * Remove SpinnerColumn from progress bar configurations and replace with _MofNCompleteColumn for improved display of completed tasks. * Quiet OCR errors * Update logging messages for jurisdiction count to use pluralization * Replace `UserWarning` with `COMPASSWarning` in warning messages for ordinance extraction and county utilities * Add warnings filter to suppress warnings during processing in CLI
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Improve prompt on county validators. This is mainly intended to help with #34 and #39.