FlexMeasures / flexmeasures / 36719204342

86%

Build Type

push

github

Committed by web-flow

Commit Message

Let the scheduler clean its inputs with bounds on a sensor reference (#2561)

* Let a forecast target name the sources it trains on

The three regressor options accept a source-filtered sensor reference, so a
regressor can say which of the sources recording on a sensor it reads. The
target sensor took a bare ID only, so a sensor that several sources report on
was trained on whichever of them won each event, with no way to say which one
holds the truth.

The target now accepts the same reference. A bare ID keeps behaving as before:
every source is trained on except forecasters, which are left out so that a
forecaster does not learn from its own forecasts. A reference replaces that
default rather than adding to it, so pass exclude-source-types yourself to keep
forecasters out alongside another filter.

Forecasts are still recorded on the sensor itself, never on a source-filtered
view of it, so the output sensor unwraps a referenced target. The reference
survives serialization into a queued job and storage on an automation.

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* docs: changelog entry for the forecast target source filter

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* data/models/forecasting: extract reusable bound parsing and application

Context:
- Clipping and snapping are about to be applied to forecaster inputs as well as
  to forecast output, so the logic can no longer live inside the output path.

Change:
- Split parse_bounds and apply_bounds_to_values out of apply_forecast_post_processing,
  keeping snap-then-clip order and the [first, second) interval semantics.
- Let error messages carry a label, so input bounds can name the sensor at fault.
- Move _is_parseable_quantity here from the schema, where the input reference
  schema can reach it too without importing the pipeline schema.

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* data/models/forecasting: hand the... (continued)

Coverage Stats

39 of 40 new or added lines in 3 files covered. (97.5%)

1 existing line in 1 file now uncovered.

20717 of 24032 relevant lines covered (86.21%)

0.86 hits per line