FlexMeasures / flexmeasures / 18964786904

Pull #1546

github

Add max training period param to cap training window (#1741)

* feat: add max-training-period option to CLI command

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

* feat: add max_training_period field to ForecasterParametersSchema and enforce its limit in validation

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

* feat: add logging for train-period exceeding max-training-period in ForecasterParametersSchema validation

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

* test: add test to verify logging and capping of train_period exceeding max_training_period

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

* style: run pre-commit

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

---------

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>
Signed-off-by: Mohamed Belhsan Hmida <149331360+BelhsanHmida@users.noreply.github.com>

1352 of 1972 branches covered (68.56%)

Branch coverage included in aggregate %.

673 of 827 new or added lines in 21 files covered. (81.38%)

9977 of 13437 relevant lines covered (74.25%)

2.97 hits per line