FlexMeasures / flexmeasures / 28507820553
81%
main: 83%

LAST BUILD BRANCH: test-1947-multi-session-same-sensor
DEFAULT BRANCH: main
Ran
Jobs 1
Files 166
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 80.671% (-0.2%) from 80.821%
28507820553

Pull #2172

github

web-flow 
Feature: flex-context per commodity as list (#2235)

* feat: support passing flex-context as a list (one flex-context per commodity)

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

* fix: set default flex-context commodity to electricity

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

* fix: preserve field order in case schema is made OpenAPI compatible

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

* feat: reduce documented nesting when defining a flex-context per commodity

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

* dev: add todos

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

* style: flake8

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

* scheduling: complete schema refactoring per PR #2235

- Add SensorReferenceSchema import
- Override relax_constraints default to True in CommodityFlexContextSchema
- Remove duplicate breach price and relax fields from FlexContextSchema
- Remove duplicate set_default_breach_prices method from FlexContextSchema

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>
Signed-off-by: F.N. Claessen <claessen@seita.nl>

* tests: add comprehensive tests for schema refactoring

- Test aggregate-consumption and aggregate-production fields
- Test SharedSchema fields accessible in FlexContextSchema
- Test CommodityFlexContextSchema relax_constraints defaults to True
- Test shared currency logic for flex-context listings
- Test breach prices in both schemas
- Add noqa comment for existing unused variable

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>

* scheduling: add shared currency validation for commodity contexts

- Add validator to check prices share same currency across all commodity contexts
- Fix test data to use actual fixture sensor names
- All new tests now passing

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>

* docs: add aggregate fields to documentation and update tests

- Add aggregate-consumption and aggregate-production to schedul... (continued)
Pull Request #2172: feat: Split flex-context settings by commodity for dynamic capacity scheduling

242 of 259 new or added lines in 12 files covered. (93.44%)

37 existing lines in 3 files now uncovered.

13464 of 16690 relevant lines covered (80.67%)

0.81 hits per line

Uncovered Changes

Lines Coverage ∆ File
9
88.47
 0.15% flexmeasures/data/models/planning/storage.py
5
93.38
 -0.52% flexmeasures/data/schemas/scheduling/__init__.py
2
89.39
 -0.14% flexmeasures/data/models/planning/linear_optimization.py
1
74.56
 0.98% flexmeasures/utils/coding_utils.py

Coverage Regressions

Lines Coverage ∆ File
31
76.19
 -17.01% flexmeasures/data/schemas/scheduling/storage.py
5
88.24
 -9.68% flexmeasures/api/common/schemas/utils.py
1
88.47
 0.15% flexmeasures/data/models/planning/storage.py
Jobs
ID Job ID Ran Files Coverage
1 28507820553.1 166
80.67
 GitHub Action Run
Source Files on build 28507820553