Small wind extraction updates#319
Merged
Merged
Conversation
ppinchuk
added
enhancement
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the codebase to rename "accessory wind" to "small wind" throughout the project, reflecting updated terminology for non-commercial wind energy systems. The changes also include improvements to prompts, documentation updates, and new setback features.
Key Changes:
- Renamed all "accessory wind" references to "small wind" across classes, functions, variables, and documentation
- Updated terminology in prompts to "small, medium, or non-commercial wind energy systems"
- Added "public conservation lands" as a new setback feature
- Enhanced prompts with better formatting and clarification about system exclusions
Reviewed Changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| compass/utilities/io.py | Updated docstring to correct class reference and clarify document loading behavior |
| compass/utilities/finalize.py | Changed tech parameter documentation from "accessory wind" to "small wind" |
| compass/scripts/process.py | Updated imports and references from AccessoryWind* to SmallWind* classes |
| compass/extraction/wind/parse.py | Added formatting to SYSTEM_SIZE_REMINDER with tech parameter |
| compass/extraction/solar/parse.py | Added formatting to SYSTEM_SIZE_REMINDER with tech parameter and fixed syntax error |
| compass/extraction/small_wind/parse.py | Renamed classes/variables from AccessoryWind to SmallWind, updated prompts and system messages, added public conservation lands feature |
| compass/extraction/small_wind/ordinance.py | Renamed classes/variables, updated prompts with new terminology and exclusion types |
| compass/extraction/small_wind/graphs.py | Updated prompts and node names to use small wind terminology |
| compass/extraction/small_wind/init.py | Updated exports to use SmallWind naming |
| compass/extraction/features.py | Added public conservation lands setback feature with clarifications |
| compass/common/base.py | Enhanced summary prompts and added new "explicitly not regulated" (ENR) graph branch |
Comments suppressed due to low confidence (6)
compass/extraction/small_wind/parse.py:38
- Incorrect punctuation: periods should be commas. The text 'small. medium' should be 'small, medium' to properly list the terms.
compass/extraction/small_wind/parse.py:45 - Corrected spelling of 'parsel' to 'parcel'.
compass/extraction/small_wind/ordinance.py:261 - Incorrect punctuation: periods should be commas. The text 'small. medium' should be 'small, medium' to properly list the terms.
compass/extraction/small_wind/ordinance.py:400 - Incorrect punctuation: periods should be commas. The text 'small. medium' should be 'small, medium' to properly list the terms.
compass/extraction/small_wind/ordinance.py:413 - Incorrect punctuation: periods should be commas. The text 'small. medium' should be 'small, medium' to properly list the terms.
compass/extraction/small_wind/init.py:21 - Implicit string concatenation. Maybe missing a comma?
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
rajeee
pushed a commit
that referenced
this pull request
May 27, 2026
* Update docstrings * clarify comment * Add missing tasks * Shift focus to small wind energy systems * Update summary instructions * Update for small systems * Move to small wind * Format system size reminder prompt * Track setbacks from public conservation lands * Update prompts for small wind * Add check for explicitly not-regulated ordinances * accessory -> small * Rename module * Minor typo fix * Update link
This file contains hidden or 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
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.
Minor updates to focus small wind extraction