-
Notifications
You must be signed in to change notification settings - Fork 46
Feat/schedule resolution #1857
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Feat/schedule resolution #1857
Conversation
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
…resolution Signed-off-by: F.N. Claessen <felix@seita.nl>
…tion Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
…l on the tick Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Documentation build overview
Show files changed (4 files in total): 📝 4 modified | ➕ 0 added | ➖ 0 deleted
|
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
…g job to sequential scheduling jobs Signed-off-by: F.N. Claessen <felix@seita.nl>
…lution Signed-off-by: F.N. Claessen <felix@seita.nl>
…set scheduling Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
This reverts commit e252d27. Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
nhoening
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested this in the CLI via the first tutorial and ran into a problem.
I used docker compose and ran the script, only with --resolution PT2H added to the flexmeasures add schedule command.
The function ensure_prices_are_not_empty() complained (see below), I suspect because get_continuous_series_sensor_or_quantity() cannot get the data if the resolution passed to it doesn't match the sensor resolution? I did not pursue that until the end (TimedBelied.search()).
File "/usr/local/lib/python3.12/dist-packages/flexmeasures/cli/data_add.py", line 1444, in add_schedule
success = make_schedule(
^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/dist-packages/flexmeasures/data/services/scheduling.py", line 584, in make_schedule
consumption_schedule: SchedulerOutputType = scheduler.compute()
^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/dist-packages/flexmeasures/data/models/planning/storage.py", line 1318, in compute
) = self._prepare(skip_validation=skip_validation)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/dist-packages/flexmeasures/data/models/planning/storage.py", line 187, in _prepare
ensure_prices_are_not_empty(up_deviation_prices, consumption_price)
File "/usr/local/lib/python3.12/dist-packages/flexmeasures/data/models/planning/utils.py", line 96, in ensure_prices_are_not_empty
raise UnknownPricesException(error_message)
flexmeasures.data.models.planning.exceptions.UnknownPricesException: Prices unknown for planning window. (sensor 1)
Description
resolutionwhen triggering a schedule via the API--resolutionoption in the CLI forflexmeasures add scheduledocumentation/changelog.rstLook & Feel
...
How to test
test_trigger_and_get_scheduletest_asset_trigger_and_get_scheduleFurther Improvements
Related Items
Closes #1856.
...