Review/897/refactoring#908
Merged
Flix6x merged 4 commits intofeature/planning/battery-power-capacity-as-sensorfrom Nov 24, 2023
Merged
Review/897/refactoring#908Flix6x merged 4 commits intofeature/planning/battery-power-capacity-as-sensorfrom
Flix6x merged 4 commits intofeature/planning/battery-power-capacity-as-sensorfrom
Conversation
Signed-off-by: F.N. Claessen <[email protected]>
Signed-off-by: F.N. Claessen <[email protected]>
Signed-off-by: F.N. Claessen <[email protected]>
Signed-off-by: F.N. Claessen <[email protected]>
Contributor
Author
|
Also, I feel we should be using dashed - rather than underscored - fields for serialized sensor/asset attributes in our database. This should results in a more consistent experience to API/UI users in the long run. We could of course make that a separate issue, but we are introducing two new fields in #897, so we might as well tackle those there. What do you think @victorgarcia98? If you agree, let's hold off on refactoring until merging this PR and also after considering my next post-review PR, to save us the merge conflicts. |
victorgarcia98
approved these changes
Nov 24, 2023
Contributor
You're right, this is worth its own issue. In any case, we shouldn't mix dashed and underscored in the DB. Issue #911 |
victorgarcia98
pushed a commit
that referenced
this pull request
Dec 19, 2023
* add (device) consumption capacity and production capacity as sensors Signed-off-by: Victor Garcia Reolid <[email protected]> * convert units of the power limits to the sensor units Signed-off-by: Victor Garcia Reolid <[email protected]> * add QuantityOrSensor field Signed-off-by: Victor Garcia Reolid <[email protected]> * add annotations Signed-off-by: Victor Garcia Reolid <[email protected]> * introduce QuantityorSensor schema Signed-off-by: Victor Garcia Reolid <[email protected]> * add from __future__ import annotations Signed-off-by: Victor Garcia Reolid <[email protected]> * remove discarded revision Signed-off-by: Victor Garcia Reolid <[email protected]> * add target unit Signed-off-by: Victor Garcia Reolid <[email protected]> * change exception type to FMValidationError Signed-off-by: Victor Garcia Reolid <[email protected]> * add explicity default value Signed-off-by: Victor Garcia Reolid <[email protected]> * split into two functions Signed-off-by: Victor Garcia Reolid <[email protected]> * Review/897/refactoring (#908) * refactor: fix spelling Signed-off-by: F.N. Claessen <[email protected]> * style: missing type annotation Signed-off-by: F.N. Claessen <[email protected]> * refactor: method and variable renaming Signed-off-by: F.N. Claessen <[email protected]> * refactor: add test case explanations Signed-off-by: F.N. Claessen <[email protected]> --------- Signed-off-by: F.N. Claessen <[email protected]> * Review/897/distinguish fallback value and max value (#909) * refactor: fix spelling Signed-off-by: F.N. Claessen <[email protected]> * style: missing type annotation Signed-off-by: F.N. Claessen <[email protected]> * refactor: method and variable renaming Signed-off-by: F.N. Claessen <[email protected]> * refactor: add test case explanations Signed-off-by: F.N. Claessen <[email protected]> * fix: separate logic for falling back on a default attribute and applying a maximum capacity limit Signed-off-by: F.N. Claessen <[email protected]> * fix: fill gaps in capacity sensor data using the max_value rather than with the fallback Signed-off-by: F.N. Claessen <[email protected]> * fix: type annotation (see https://docs.python.org/3/library/typing.html#typing.Optional) Signed-off-by: F.N. Claessen <[email protected]> * refactor: clearly separate steps of falling back and performing a nanmin Signed-off-by: F.N. Claessen <[email protected]> * refactor: simplify and have get_quantity_from_attribute return a Quantity, as its name already suggested Signed-off-by: F.N. Claessen <[email protected]> * refactor: switch argument order Signed-off-by: F.N. Claessen <[email protected]> * refactor: update docstring Signed-off-by: F.N. Claessen <[email protected]> * style: flake8 Signed-off-by: F.N. Claessen <[email protected]> --------- Signed-off-by: F.N. Claessen <[email protected]> * fix test Signed-off-by: Victor Garcia Reolid <[email protected]> * add changelog entry Signed-off-by: Victor Garcia Reolid <[email protected]> * update changelog Signed-off-by: Victor Garcia Reolid <[email protected]> --------- Signed-off-by: Victor Garcia Reolid <[email protected]> Signed-off-by: F.N. Claessen <[email protected]> Co-authored-by: Felix Claessen <[email protected]>
victorgarcia98
pushed a commit
that referenced
this pull request
Dec 19, 2023
* add (device) consumption capacity and production capacity as sensors Signed-off-by: Victor Garcia Reolid <[email protected]> * convert units of the power limits to the sensor units Signed-off-by: Victor Garcia Reolid <[email protected]> * add QuantityOrSensor field Signed-off-by: Victor Garcia Reolid <[email protected]> * add annotations Signed-off-by: Victor Garcia Reolid <[email protected]> * introduce QuantityorSensor schema Signed-off-by: Victor Garcia Reolid <[email protected]> * add from __future__ import annotations Signed-off-by: Victor Garcia Reolid <[email protected]> * remove discarded revision Signed-off-by: Victor Garcia Reolid <[email protected]> * add target unit Signed-off-by: Victor Garcia Reolid <[email protected]> * change exception type to FMValidationError Signed-off-by: Victor Garcia Reolid <[email protected]> * add explicity default value Signed-off-by: Victor Garcia Reolid <[email protected]> * split into two functions Signed-off-by: Victor Garcia Reolid <[email protected]> * add usage forecast Signed-off-by: Victor Garcia Reolid <[email protected]> * add test Signed-off-by: Victor Garcia Reolid <[email protected]> * Review/897/refactoring (#908) * refactor: fix spelling Signed-off-by: F.N. Claessen <[email protected]> * style: missing type annotation Signed-off-by: F.N. Claessen <[email protected]> * refactor: method and variable renaming Signed-off-by: F.N. Claessen <[email protected]> * refactor: add test case explanations Signed-off-by: F.N. Claessen <[email protected]> --------- Signed-off-by: F.N. Claessen <[email protected]> * use resolution to convert energy series to the target event resolution Signed-off-by: Victor Garcia Reolid <[email protected]> * simplify conftest Signed-off-by: Victor Garcia Reolid <[email protected]> * Review/897/distinguish fallback value and max value (#909) * refactor: fix spelling Signed-off-by: F.N. Claessen <[email protected]> * style: missing type annotation Signed-off-by: F.N. Claessen <[email protected]> * refactor: method and variable renaming Signed-off-by: F.N. Claessen <[email protected]> * refactor: add test case explanations Signed-off-by: F.N. Claessen <[email protected]> * fix: separate logic for falling back on a default attribute and applying a maximum capacity limit Signed-off-by: F.N. Claessen <[email protected]> * fix: fill gaps in capacity sensor data using the max_value rather than with the fallback Signed-off-by: F.N. Claessen <[email protected]> * fix: type annotation (see https://docs.python.org/3/library/typing.html#typing.Optional) Signed-off-by: F.N. Claessen <[email protected]> * refactor: clearly separate steps of falling back and performing a nanmin Signed-off-by: F.N. Claessen <[email protected]> * refactor: simplify and have get_quantity_from_attribute return a Quantity, as its name already suggested Signed-off-by: F.N. Claessen <[email protected]> * refactor: switch argument order Signed-off-by: F.N. Claessen <[email protected]> * refactor: update docstring Signed-off-by: F.N. Claessen <[email protected]> * style: flake8 Signed-off-by: F.N. Claessen <[email protected]> --------- Signed-off-by: F.N. Claessen <[email protected]> * fix test Signed-off-by: Victor Garcia Reolid <[email protected]> * add changelog entry Signed-off-by: Victor Garcia Reolid <[email protected]> * improve tests and implement consumption_is_positive Signed-off-by: Victor Garcia Reolid <[email protected]> * add resolution factor to convert from energy units to power units Signed-off-by: Victor Garcia Reolid <[email protected]> * fix test + add new case Signed-off-by: Victor Garcia Reolid <[email protected]> * split stock delta into stock gain and stock usage Signed-off-by: Victor Garcia Reolid <[email protected]> * update changelog Signed-off-by: Victor Garcia Reolid <[email protected]> * switch to power units Signed-off-by: Victor Garcia Reolid <[email protected]> * improve comments Signed-off-by: Victor Garcia Reolid <[email protected]> * add changelog Signed-off-by: Victor Garcia Reolid <[email protected]> * Update changelog.rst Fix typo (*los componentes*) and swap out some changelog entries between Features and Infrastructure. Signed-off-by: Felix Claessen <[email protected]> * Apply black Signed-off-by: Victor Garcia Reolid <[email protected]> --------- Signed-off-by: Victor Garcia Reolid <[email protected]> Signed-off-by: F.N. Claessen <[email protected]> Signed-off-by: Victor <[email protected]> Signed-off-by: Felix Claessen <[email protected]> Co-authored-by: Felix Claessen <[email protected]>
Flix6x
added a commit
that referenced
this pull request
Dec 21, 2023
* add (device) consumption capacity and production capacity as sensors Signed-off-by: Victor Garcia Reolid <[email protected]> * convert units of the power limits to the sensor units Signed-off-by: Victor Garcia Reolid <[email protected]> * add QuantityOrSensor field Signed-off-by: Victor Garcia Reolid <[email protected]> * add annotations Signed-off-by: Victor Garcia Reolid <[email protected]> * introduce QuantityorSensor schema Signed-off-by: Victor Garcia Reolid <[email protected]> * add from __future__ import annotations Signed-off-by: Victor Garcia Reolid <[email protected]> * remove discarded revision Signed-off-by: Victor Garcia Reolid <[email protected]> * add target unit Signed-off-by: Victor Garcia Reolid <[email protected]> * change exception type to FMValidationError Signed-off-by: Victor Garcia Reolid <[email protected]> * add explicity default value Signed-off-by: Victor Garcia Reolid <[email protected]> * split into two functions Signed-off-by: Victor Garcia Reolid <[email protected]> * Review/897/refactoring (#908) * refactor: fix spelling Signed-off-by: F.N. Claessen <[email protected]> * style: missing type annotation Signed-off-by: F.N. Claessen <[email protected]> * refactor: method and variable renaming Signed-off-by: F.N. Claessen <[email protected]> * refactor: add test case explanations Signed-off-by: F.N. Claessen <[email protected]> --------- Signed-off-by: F.N. Claessen <[email protected]> * Review/897/distinguish fallback value and max value (#909) * refactor: fix spelling Signed-off-by: F.N. Claessen <[email protected]> * style: missing type annotation Signed-off-by: F.N. Claessen <[email protected]> * refactor: method and variable renaming Signed-off-by: F.N. Claessen <[email protected]> * refactor: add test case explanations Signed-off-by: F.N. Claessen <[email protected]> * fix: separate logic for falling back on a default attribute and applying a maximum capacity limit Signed-off-by: F.N. Claessen <[email protected]> * fix: fill gaps in capacity sensor data using the max_value rather than with the fallback Signed-off-by: F.N. Claessen <[email protected]> * fix: type annotation (see https://docs.python.org/3/library/typing.html#typing.Optional) Signed-off-by: F.N. Claessen <[email protected]> * refactor: clearly separate steps of falling back and performing a nanmin Signed-off-by: F.N. Claessen <[email protected]> * refactor: simplify and have get_quantity_from_attribute return a Quantity, as its name already suggested Signed-off-by: F.N. Claessen <[email protected]> * refactor: switch argument order Signed-off-by: F.N. Claessen <[email protected]> * refactor: update docstring Signed-off-by: F.N. Claessen <[email protected]> * style: flake8 Signed-off-by: F.N. Claessen <[email protected]> --------- Signed-off-by: F.N. Claessen <[email protected]> * fix test Signed-off-by: Victor Garcia Reolid <[email protected]> * add changelog entry Signed-off-by: Victor Garcia Reolid <[email protected]> * update changelog Signed-off-by: Victor Garcia Reolid <[email protected]> * add charge and discharge efficiencies Signed-off-by: Victor Garcia Reolid <[email protected]> * fix test Signed-off-by: Victor Garcia Reolid <[email protected]> * add extra validation and use 100 for the missing values instead of the roundtrip Signed-off-by: Victor Garcia Reolid <[email protected]> * fix validation rules Signed-off-by: Victor Garcia Reolid <[email protected]> * check roundtrip efficiency Signed-off-by: Victor Garcia Reolid <[email protected]> * add documentation Signed-off-by: Victor Garcia Reolid <[email protected]> --------- Signed-off-by: Victor Garcia Reolid <[email protected]> Signed-off-by: F.N. Claessen <[email protected]> Co-authored-by: Felix Claessen <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I split my post-review work into two parts. This one deals with refactoring only.