-
-
Notifications
You must be signed in to change notification settings - Fork 55
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
Eval: missing real generation_power values #110
Comments
Thanks @JasonFengGit for this We'll have to think how to perhaps create a new test dataset that doesnt have any missing generation values |
We could filter out timestamps with missing values, but that would introduce some biases that are hard to analyze. |
I think we could filter out the missing ones, and introduce new ones. What bias' were you thinking about? |
For example, the missing values might be due to similar reasons and could share some patterns that are either easier or harder to predict, thereby making the evaluation biased. |
ah I see, from what I've seen, there are normally quite random as they are all random pv panels throughout the UK. But we can check this |
Oh OK! That would make it easier. |
@JasonFengGit Nice spot! thanks for this |
Describe the bug
In evaluation, some of the real/expected values of generation_power are missing.
To Reproduce
Steps to reproduce the behavior:
python scripts/run_evaluation.py
with the followingtestset.csv
(a small test to illustrate the bug):results.csv
in the generation_power columnsExample
results.csv
:Expected behavior
No missing values (or maybe some fallbacks to handle missing values).
The text was updated successfully, but these errors were encountered: