fix default genrator mapping file#66
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the default generator-to-category mapping used by PowerAnalytics to classify natural-gas generators into the intended categories.
Changes:
- Adjust
NG-CTmapping to use prime moverGTforNATURAL_GAS. - Expand
NG-CCmapping to include additional prime movers (CA,CT) forNATURAL_GAS.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot Open a PR which fixes the expected test values so the tests pass. The value changes are expected given the change of generator mappings. |
|
@m-bossart I've opened a new pull request, #67, to work on those changes. Once the pull request is ready, I'll request review from you. |
Agent-Logs-Url: https://github.com/NREL-Sienna/PowerAnalytics.jl/sessions/21d938ee-1820-4f32-8aa1-947ad23f62af Co-authored-by: m-bossart <67015312+m-bossart@users.noreply.github.com>
Fix test expected values after generator mapping update
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #66 +/- ##
=======================================
Coverage 88.04% 88.04%
=======================================
Files 7 7
Lines 669 669
=======================================
Hits 589 589
Misses 80 80
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
closed in favor of #72 |
No description provided.