FlexMeasures / flexmeasures / 20172916137

86%

Build Type

push

github

Committed by web-flow

Commit Message

Fix: duplicate beliefs by storing non-floored belief_time in forecasting pipeline (#1855)

* Update flexmeasures/data/models/forecasting/pipelines/base.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Mohamed Belhsan Hmida <149331360+BelhsanHmida@users.noreply.github.com>

* Update flexmeasures/data/schemas/forecasting/pipeline.py

Co-authored-by: Felix Claessen <30658763+Flix6x@users.noreply.github.com>
Signed-off-by: Mohamed Belhsan Hmida <149331360+BelhsanHmida@users.noreply.github.com>

* chore: reassign changelog entry to v0.30.2

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

* feat: Add save_belief_time parameter to BasePipeline to be used as non floored belief_time when saving forecasts

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

* docs: Add comment to clarify purpose of save_belief_time in BasePipeline

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

* feat: Update belief_time handling to use save_belief_time

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

* feat: Add save_belief_time parameter to PredictPipeline for non-floored belief time

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

* feat: Add save_belief_time parameter to TrainPipeline for non-floored belief time

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

* feat: Pass save_belief_time parameter to TrainPredictPipeline

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

* feat: Update predict_start logic to conditionally set save_belief_time

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

* fix: Correct belief_time attribute to belief_times in train_predict_pipeline

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

* Update flexmeasures/data/schemas/forecasting/pipeline.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Mohamed B... (continued)

Coverage Stats

1394 of 2051 branches covered (67.97%)

Branch coverage included in aggregate %.

9 of 10 new or added lines in 2 files covered. (90.0%)

39 existing lines in 4 files now uncovered.

10312 of 13879 relevant lines covered (74.3%)

5.94 hits per line