-
Notifications
You must be signed in to change notification settings - Fork 38
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
feat: add AggregatorReporter
#712
Conversation
…Error Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
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.
Very nice to have. Are you sure you don't want this to subclass from PandasReporter instead?
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
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.
Looks like this can even be shipped with 0.14. I added a changelog and moved the relevant entry to the Infrastructure / Support category. I also added an experimental disclaimer, because we are planning to still refactor these classes soon (see #662).
So can this be merged? |
Yes, but let's give @victorgarcia98 a chance to review my commits first. |
Thanks! Nice the addition of the experimental warning. |
Description
This PR adds the class
AggregatorReporter
to compute the aggregation (SUM
orMEAN
) of different sources' into a single one. This reporter allows to add the aggregate energy flows, costs, emissions, etc... of a system in a time period (e.g hourly).Details:
source
a required parameter inbeliefs_search_configs
.weights
for the different alias. This can be used to convert a consumption (in absolute value) to a negative flow or to convert a current into power (i.emethod
can be set toSUM
orMEAN
to chose the aggregation method.Look & Feel
Testing Steps
Test for
MEAN
andSUM
cases in flexmeasures/data/models/reporting/tests/test_aggregator.py.