FlexMeasures / flexmeasures / 29092452276

81%

push

github

Fix/detached forecasting sensors (#2035)

* fix: enhance job ID return structure in TrainPredictPipeline

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

* fix: update forecasting job return structure in SensorAPI

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

* fix: update job fetching logic in test_train_predict_pipeline

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

* chore: remove commented-out breakpoint in test_forecasting.py

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

* fix: as_job is no longer in  parameters

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

* fix: update job count retrieval in add_forecast function

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

* fix: add connection queue to fetch job

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

* style: black

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

* docs: changelog entry

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

* feat: check if wrap-up job actually finished rather than failed

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

* feat: add test case for 2 cycles, yielding 2 jobs and a wrap-up job

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

* dev: comment out failing assert, which needs to be investgated and updated

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

* refactor: move checking the status of the wrap-up job to where it matters

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

* fix: use job ID itself in case the returned job is the one existing cycle job

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

* fix: add db.commit before forecasting jobs are created

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

* dev: uncomment test assertion statement

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

* Test(feat): search all beliefs forecasts save... (continued)

77 of 83 new or added lines in 2 files covered. (92.77%)

14216 of 17499 relevant lines covered (81.24%)

0.81 hits per line