FlexMeasures / flexmeasures / 22023184253 / 2

78%

main: 78%

push

github

Feat/forecasting parameter single source (#1955)

* feat(cli): add decorator to add CLI options from Marshmallow schema fields

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

* feat: add cli option schema fields metadata.

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

* dev: integrate ForecasterParametersSchema and add CLI options from schema using add_cli_options_from_schema decorator

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

* chore: remove commented out max-training-period from cli params

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

* chore: remove commented out sensor from cli params

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

* chore: remove commented out params which have same descriptions in schema

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

* style: adjust docstring formatting in add_cli_options_from_schema function

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

* chore: update max-forecast-horizon description to use ISO 8601 format to show input format

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

* chore: update descriptions for future, past, and general regressors in ForecasterParametersSchema. maybe future and past regressors isn't clear for users

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

* feat: support multiple option names and aliases in add_cli_options_from_schema function

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

* feat: add aliases for CLI options in ForecasterParametersSchema

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

* chore: remove commented-out CLI options for training period and retrain frequency in data_add.py

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

* chore: clean up dev comments for params that are going to sta... (continued)

1633 of 2157 branches covered (75.71%)

Branch coverage included in aggregate %.

22482 of 28798 relevant lines covered (78.07%)

0.78 hits per line