FlexMeasures / flexmeasures / 29406512349

82%

main: 82%

Build Type

Pull #2321

github

Committed by Flix6x

Commit Message

fix: fail fast on conflicting stock parameters (review)

Address review comments on PR #2321:

- Raise a ValidationError when more than one flex-model entry defines
  state-of-charge parameters for the same stock, instead of silently
  letting the last entry win (stricter than the historical behavior).
- Reflow attribute comments to natural clause breaks, clarify that the
  flex_model None check is a precaution for inflexible devices, and note
  why inflexible devices always have a power sensor.
- Word choice: scheduling features (not code sites) re-derived entry
  kinds; pass the sensor name as a keyword argument in the test helper.

Signed-off-by: F.N. Claessen <felix@seita.nl>

Pull Request Pull Request #2321: Track devices via a typed device inventory in the StorageScheduler

Coverage Stats

175 of 190 new or added lines in 5 files covered. (92.11%)

19 existing lines in 2 files now uncovered.

14671 of 17995 relevant lines covered (81.53%)

0.82 hits per line