Skip to content

Sub area stats fix - #583

Merged
ppinchuk merged 3 commits into
mainfrom
pp/sub_area_stats_fix
Aug 11, 2026
Merged

Sub area stats fix#583
ppinchuk merged 3 commits into
mainfrom
pp/sub_area_stats_fix

Conversation

@ppinchuk

Copy link
Copy Markdown
Collaborator

Sub area calc was not quite using the same mask as a generation supply curve point, so this patch fixes and edge case where they diverge

@ppinchuk ppinchuk self-assigned this Aug 11, 2026
@ppinchuk ppinchuk added bugfix Fixes a bug p-high Priority: high topic-aggregation Issues/pull requests related to supply curve aggregation labels Aug 11, 2026
@ppinchuk ppinchuk linked an issue Aug 11, 2026 that may be closed by this pull request
3 tasks
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@aaa5056). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #583   +/-   ##
=======================================
  Coverage        ?   79.48%           
=======================================
  Files           ?       64           
  Lines           ?    11811           
  Branches        ?     1682           
=======================================
  Hits            ?     9388           
  Misses          ?     1901           
  Partials        ?      522           
Flag Coverage Δ
unittests 79.48% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ppinchuk
ppinchuk merged commit 4dc2251 into main Aug 11, 2026
15 of 16 checks passed
github-actions Bot pushed a commit that referenced this pull request Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixes a bug p-high Priority: high topic-aggregation Issues/pull requests related to supply curve aggregation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sub area stats raises assertion error

2 participants