-
Notifications
You must be signed in to change notification settings - Fork 39
Description
As STEF user
I want to merge the BacktestConfig and the BacktestForecasterConfig
So that I can more easily configure how the backtest is run.
🌍 Background
Important
Changes should be done by forking OpenSTEF 4.0.0 tracking branch (release/v4.0.0) and then make a PR back onto release/v4.0.0 branch.
Please read the contributing guide before you start.
BacktestForecasterConfig:
openstef/packages/openstef-beam/src/openstef_beam/backtesting/backtest_forecaster/mixins.py
Line 24 in b6c13fa
| class BacktestForecasterConfig(BaseConfig): |
(Used in
| backtest_config = BacktestForecasterConfig( |
BacktestConfig:
https://github.com/OpenSTEF/openstef/blob/release/v4.0.0/packages/openstef-beam/src/openstef_beam/backtesting/backtest_pipeline.py#L31
❗Priority (What if we don't do this?/Are there any deadlines? etc.)
Definition of Done:
✅ Acceptance criteria
- The two configs are merged
- Benchmark example is adjusted to use this new config
📄 Documentation criteria:
Interfaces with other products and services
Update relevant confluence pages
🧪 Test criteria:
Unit tests written and passed
⌛ Dependencies:
N/A
🚀 Releasing:
N/A
Other information:
Metadata
Metadata
Assignees
Labels
Type
Projects
Status