FlexMeasures / flexmeasures / 18964786904 / 1

74%

main: 78%

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 %.

19954 of 26874 relevant lines covered (74.25%)

0.74 hits per line