FlexMeasures / flexmeasures / 30673063886

83%

main: 85%

Build Type

Pull #2295

github

Committed by Flix6x

Commit Message

fix: pin a scoped commitment's commodity to its scoped devices; tests + comment tidy

Addresses Copilot's review on #2295:
- A scoped commitment now takes its commodity from the (single) commodity of its
  scoped devices, overriding the schema's electricity default, so its cost is not
  misattributed to the wrong commodity.
- Add tests for the commodity pinning and for the 'scope matches no device -> warn
  and bind nothing' path.
- Fix the sensor-scoped test docstring (it mentioned a battery the test never had).
- Reflow the CommitmentSchema scoping comment to break only after punctuation.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TaMepbxRYzJxQKFtu6Hq4p
Signed-off-by: F.N. Claessen <felix@seita.nl>

Pull Request Pull Request #2295: Commitments can be scoped to specific sensors, binding their aggregate flow

Coverage Stats

35 of 36 new or added lines in 2 files covered. (97.22%)

131 existing lines in 2 files now uncovered.

16341 of 19618 relevant lines covered (83.3%)

0.83 hits per line