Skip to content
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

Interpret SoC constraints defined as non instantaneous sensors #1012

Conversation

Flix6x
Copy link
Contributor

@Flix6x Flix6x commented Mar 12, 2024

Description

I think this is the best I can do without addressing the distinction between resolution and frequency pretty much from the ground up in timely-beliefs, which would be a bigger project.

Related Items

Ahmad-Wahid and others added 5 commits March 11, 2024 18:11
…sors (n) (#998)

* use scalars instead of scalar having limit constraint

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* update the test_closest_sensor, to test multiple sensor as params

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* update the docstring

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

---------

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
…1003)

* add --offspring flag to command flexmeasures delete beliefs

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add docstring

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* fix: include offspring assets in prompt and done messages

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: less alarming message in case a time window was passed

Signed-off-by: F.N. Claessen <felix@seita.nl>

* Improve processing time for deleting beliefs via CLI (#1005)

* speed up deleting beliefs via CLI

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* docs: changelog entry

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: typo

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: F.N. Claessen <felix@seita.nl>

* fix: CLI message

Signed-off-by: F.N. Claessen <felix@seita.nl>

* docs: changelog entry

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Co-authored-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: Ahmad Wahid <59763365+Ahmad-Wahid@users.noreply.github.com>
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>
@Flix6x Flix6x self-assigned this Mar 12, 2024
Copy link
Contributor

@victorgarcia98 victorgarcia98 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

I really like how the boundary_policy parameter is exposed to the function get_continuous_series_sensor_or_quantity.

Other than that, I have a comment regarding the inclusion of the left interval instead of the right one. As far as I understand, the SOC variable of the scheduler represents the value at the end of the period. If that's the case, I think we should include the interval before instead of the following one.

nhoening and others added 8 commits March 13, 2024 17:55
* introduce FLEXMEASURES_FORCE_HTTPS so we can control better when we want the app to answer to https. Apply in the internal API to solve a load balancer situation.

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* add warning to consider this setting before upgrading

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* build the URL right in the beginning of internal view functions

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

---------

Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
* Update db setup text in toy example setup

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Fix commit message Signed-off-by: Nikolai Rozanov <nickolay@seita.nl>

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Fix commit message

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Make runnable on macos + update docs accordingly

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Post review fixes. Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Post review fixes

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Add command for highs macOS installation

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Enhance test_closest_sensor to Support Variable Number of Closest Sensors (n) (#998)

* use scalars instead of scalar having limit constraint

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* update the test_closest_sensor, to test multiple sensor as params

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* update the docstring

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

---------

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* [CLI] add `--offspring` flag to command flexmeasures delete beliefs (#1003)

* add --offspring flag to command flexmeasures delete beliefs

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add docstring

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* fix: include offspring assets in prompt and done messages

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: less alarming message in case a time window was passed

Signed-off-by: F.N. Claessen <felix@seita.nl>

* Improve processing time for deleting beliefs via CLI (#1005)

* speed up deleting beliefs via CLI

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* docs: changelog entry

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: typo

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: F.N. Claessen <felix@seita.nl>

* fix: CLI message

Signed-off-by: F.N. Claessen <felix@seita.nl>

* docs: changelog entry

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Co-authored-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: Ahmad Wahid <59763365+Ahmad-Wahid@users.noreply.github.com>
Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Introduce FLEXMEASURES_FORCE_HTTPS  (#1008)

* introduce FLEXMEASURES_FORCE_HTTPS so we can control better when we want the app to answer to https. Apply in the internal API to solve a load balancer situation.

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* add warning to consider this setting before upgrading

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* build the URL right in the beginning of internal view functions

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

---------

Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* add forgotten changelog entry for #1008

Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Changelog message

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Changelog message fix

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* [CLI] add `--offspring` flag to command flexmeasures delete beliefs (#1003)

* add --offspring flag to command flexmeasures delete beliefs

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add docstring

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* fix: include offspring assets in prompt and done messages

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: less alarming message in case a time window was passed

Signed-off-by: F.N. Claessen <felix@seita.nl>

* Improve processing time for deleting beliefs via CLI (#1005)

* speed up deleting beliefs via CLI

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* docs: changelog entry

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: typo

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: F.N. Claessen <felix@seita.nl>

* fix: CLI message

Signed-off-by: F.N. Claessen <felix@seita.nl>

* docs: changelog entry

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Co-authored-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: Ahmad Wahid <59763365+Ahmad-Wahid@users.noreply.github.com>
Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* merge master

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

---------

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>
Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: nrozanov <50166318+nrozanov@users.noreply.github.com>
Co-authored-by: Nikolai <nrozanov@iponweb.net>
Co-authored-by: Ahmad Wahid <59763365+Ahmad-Wahid@users.noreply.github.com>
Co-authored-by: Victor <victor@seita.nl>
Co-authored-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: Nicolas Höning <nicolas@seita.nl>
…fs (#1013)

* add belief horizon field

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* using correct offset

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add `belief_horizon` field to schema

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add changelog entry

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* fix entry

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
* add flag check_output_resolution and get the minimum resolution of the input data from the min of the input sensors event resolutions instead of the output sensors.

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* ammend docstring and correct flag logic

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add field to schema

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add changelog entry

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* handle default in function argument instead

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
* Get the minimum resolution of the input data from the min of the input sensors event resolutions instead of the output sensors.

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add changelog entry

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
* convert units to MW

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add changelog entry

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* update docstring.

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add inflexible devices with different units

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
* use same trailing slash policy as in main API (none)

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* do not add slashes when calling dev API, either

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* align asset API endpoints for chart data with no trailing slash policy, as well

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* align UI Crud endpoints with no trailing slash policy, as well

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* fix test

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* align two more links

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* add changelog entry & warning

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* implement option A1 from review discussion + add comments for later

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* remove changelog warning after option A1 was implemented

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

---------

Signed-off-by: Nicolas Höning <nicolas@seita.nl>
@victorgarcia98
Copy link
Contributor

Could you please change the version of timely-beliefs to 2.2.0 (I think that one includes the new resampling strategy)? I think this will fix the image build process error. Also, I'd like to see if the test pass.

@Flix6x Flix6x marked this pull request as ready for review March 22, 2024 07:47
@Flix6x
Copy link
Contributor Author

Flix6x commented Mar 22, 2024

Done. (It was already changed, so I just had to rerun the failed check. The remaining checks that fail are failing on coveralls.)

@victorgarcia98 victorgarcia98 self-requested a review March 22, 2024 13:54
Copy link
Contributor

@victorgarcia98 victorgarcia98 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logic-wise, I don't see any flaw.

flexmeasures/data/models/planning/utils.py Show resolved Hide resolved
flexmeasures/data/models/planning/utils.py Outdated Show resolved Hide resolved
flexmeasures/data/models/planning/utils.py Show resolved Hide resolved
flexmeasures/data/models/planning/utils.py Show resolved Hide resolved
flexmeasures/data/models/planning/tests/test_solver.py Outdated Show resolved Hide resolved
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>
@Flix6x Flix6x merged commit 3bd457a into feature/plannign/soc-constraints-as-sensors Mar 25, 2024
7 checks passed
@Flix6x Flix6x deleted the feature/planning/soc-constraints-as-non-instantaneous-sensors branch March 25, 2024 10:30
@Flix6x Flix6x added the Data label Mar 25, 2024
Flix6x added a commit that referenced this pull request Mar 26, 2024
* add SOC constraint as Sensor

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* Use sensor SensorIDField deserialization in the QuantityOrSensor and TimedEventSchema schemas.

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* refactor code

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* Add changelog entries

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* fix typo

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* update docstring

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* Interpret SoC constraints defined as non instantaneous sensors (#1012)

* Enhance test_closest_sensor to Support Variable Number of Closest Sensors (n) (#998)

* use scalars instead of scalar having limit constraint

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* update the test_closest_sensor, to test multiple sensor as params

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* update the docstring

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

---------

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* [CLI] add `--offspring` flag to command flexmeasures delete beliefs (#1003)

* add --offspring flag to command flexmeasures delete beliefs

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add docstring

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* fix: include offspring assets in prompt and done messages

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: less alarming message in case a time window was passed

Signed-off-by: F.N. Claessen <felix@seita.nl>

* Improve processing time for deleting beliefs via CLI (#1005)

* speed up deleting beliefs via CLI

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* docs: changelog entry

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: typo

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: F.N. Claessen <felix@seita.nl>

* fix: CLI message

Signed-off-by: F.N. Claessen <felix@seita.nl>

* docs: changelog entry

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Co-authored-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: Ahmad Wahid <59763365+Ahmad-Wahid@users.noreply.github.com>

* docs: expand explanation

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: upsample and convert to instantaneous

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: better implementation and moved to timely-beliefs

Signed-off-by: F.N. Claessen <felix@seita.nl>

* Introduce FLEXMEASURES_FORCE_HTTPS  (#1008)

* introduce FLEXMEASURES_FORCE_HTTPS so we can control better when we want the app to answer to https. Apply in the internal API to solve a load balancer situation.

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* add warning to consider this setting before upgrading

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* build the URL right in the beginning of internal view functions

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

---------

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* add forgotten changelog entry for #1008

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* Make installable on MacOS (#1000)

* Update db setup text in toy example setup

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Fix commit message Signed-off-by: Nikolai Rozanov <nickolay@seita.nl>

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Fix commit message

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Make runnable on macos + update docs accordingly

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Post review fixes. Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Post review fixes

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Add command for highs macOS installation

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Enhance test_closest_sensor to Support Variable Number of Closest Sensors (n) (#998)

* use scalars instead of scalar having limit constraint

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* update the test_closest_sensor, to test multiple sensor as params

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* update the docstring

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

---------

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* [CLI] add `--offspring` flag to command flexmeasures delete beliefs (#1003)

* add --offspring flag to command flexmeasures delete beliefs

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add docstring

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* fix: include offspring assets in prompt and done messages

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: less alarming message in case a time window was passed

Signed-off-by: F.N. Claessen <felix@seita.nl>

* Improve processing time for deleting beliefs via CLI (#1005)

* speed up deleting beliefs via CLI

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* docs: changelog entry

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: typo

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: F.N. Claessen <felix@seita.nl>

* fix: CLI message

Signed-off-by: F.N. Claessen <felix@seita.nl>

* docs: changelog entry

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Co-authored-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: Ahmad Wahid <59763365+Ahmad-Wahid@users.noreply.github.com>
Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Introduce FLEXMEASURES_FORCE_HTTPS  (#1008)

* introduce FLEXMEASURES_FORCE_HTTPS so we can control better when we want the app to answer to https. Apply in the internal API to solve a load balancer situation.

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* add warning to consider this setting before upgrading

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* build the URL right in the beginning of internal view functions

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

---------

Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* add forgotten changelog entry for #1008

Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Changelog message

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* Changelog message fix

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* [CLI] add `--offspring` flag to command flexmeasures delete beliefs (#1003)

* add --offspring flag to command flexmeasures delete beliefs

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add docstring

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* fix: include offspring assets in prompt and done messages

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: less alarming message in case a time window was passed

Signed-off-by: F.N. Claessen <felix@seita.nl>

* Improve processing time for deleting beliefs via CLI (#1005)

* speed up deleting beliefs via CLI

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>

* docs: changelog entry

Signed-off-by: F.N. Claessen <felix@seita.nl>

* fix: typo

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: F.N. Claessen <felix@seita.nl>

* fix: CLI message

Signed-off-by: F.N. Claessen <felix@seita.nl>

* docs: changelog entry

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Co-authored-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: Ahmad Wahid <59763365+Ahmad-Wahid@users.noreply.github.com>
Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

* merge master

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>

---------

Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>
Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: nrozanov <50166318+nrozanov@users.noreply.github.com>
Co-authored-by: Nikolai <nrozanov@iponweb.net>
Co-authored-by: Ahmad Wahid <59763365+Ahmad-Wahid@users.noreply.github.com>
Co-authored-by: Victor <victor@seita.nl>
Co-authored-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: Nicolas Höning <nicolas@seita.nl>

* [Reporting] Define belief horizon for the PandasReporter output beliefs (#1013)

* add belief horizon field

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* using correct offset

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add `belief_horizon` field to schema

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add changelog entry

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* fix entry

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* [Reporting] skip output event resolution in `PandasReporter` (#1009)

* add flag check_output_resolution and get the minimum resolution of the input data from the min of the input sensors event resolutions instead of the output sensors.

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* ammend docstring and correct flag logic

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add field to schema

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add changelog entry

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* handle default in function argument instead

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* [Reporting] Fix `PandasReporter` input event resolution (#1010)

* Get the minimum resolution of the input data from the min of the input sensors event resolutions instead of the output sensors.

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add changelog entry

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* [Scheduling] convert the units of inflexible power sensors MW (#1007)

* convert units to MW

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add changelog entry

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* update docstring.

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add inflexible devices with different units

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* Align API endpoint policy w.r.t. trailing slash (#1014)

* use same trailing slash policy as in main API (none)

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* do not add slashes when calling dev API, either

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* align asset API endpoints for chart data with no trailing slash policy, as well

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* align UI Crud endpoints with no trailing slash policy, as well

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* fix test

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* align two more links

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* add changelog entry & warning

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* implement option A1 from review discussion + add comments for later

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* remove changelog warning after option A1 was implemented

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

---------

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* docs: fix typo and remove obsolete comment

Signed-off-by: F.N. Claessen <felix@seita.nl>

* refactor: use xfail

Signed-off-by: F.N. Claessen <felix@seita.nl>

* docs: add docstring param

Signed-off-by: F.N. Claessen <felix@seita.nl>

---------

Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>
Signed-off-by: nrozanov <50166318+nrozanov@users.noreply.github.com>
Co-authored-by: Ahmad Wahid <59763365+Ahmad-Wahid@users.noreply.github.com>
Co-authored-by: Victor <victor@seita.nl>
Co-authored-by: Nicolas Höning <nicolas@seita.nl>
Co-authored-by: nrozanov <50166318+nrozanov@users.noreply.github.com>
Co-authored-by: Nikolai <nrozanov@iponweb.net>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Ahmad Wahid <ahmedwahid16101@gmail.com>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Nikolai Rozanov <nickolay.rozanov@gmail.com>
Signed-off-by: nrozanov <50166318+nrozanov@users.noreply.github.com>
Signed-off-by: Victor <victor@seita.nl>
Co-authored-by: Ahmad Wahid <59763365+Ahmad-Wahid@users.noreply.github.com>
Co-authored-by: Felix Claessen <30658763+Flix6x@users.noreply.github.com>
Co-authored-by: Nicolas Höning <nicolas@seita.nl>
Co-authored-by: nrozanov <50166318+nrozanov@users.noreply.github.com>
Co-authored-by: Nikolai <nrozanov@iponweb.net>
@Flix6x Flix6x added this to the 0.20.0 milestone May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants