Skip to content
This repository has been archived by the owner on Jun 22, 2024. It is now read-only.

Execution on entire data as default for metric #193

Closed
marianturowski opened this issue Mar 14, 2022 · 0 comments · Fixed by #194
Closed

Execution on entire data as default for metric #193

marianturowski opened this issue Mar 14, 2022 · 0 comments · Fixed by #194
Labels
bug Something isn't working

Comments

@marianturowski
Copy link
Contributor

Describe the bug
When using a summary for a metric (e.g. MAE) on a complete data set, ": complete" is added to its key due to line 91 in metric_base.py.

To Reproduce
Steps to reproduce the behavior:
Run metric on complete data set.

Expected behavior
The default should be that metrics are executed on the entire data set. Therefore, only deviations from this default (e.g. cutouts) should be noted in the key.
@marianturowski marianturowski added the bug Something isn't working label Mar 14, 2022
@benHeid benHeid mentioned this issue Mar 15, 2022
Merged
14 tasks
@marianturowski marianturowski changed the title Execution on entrie data as default for metric Execution on entire data as default for metric Mar 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Change naming of the metric on the complete time series and adapt tests. KIT-IAI/pyWATTS
1 participant
@marianturowski