Skip to content
This repository has been archived by the owner on Sep 11, 2023. It is now read-only.

Bug: drop_pv_systems_which_produce_overnight expects normalised data, but it doesn't receive normalised data. #664

Open
JackKelly opened this issue May 17, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@JackKelly
Copy link
Member

Describe the bug
nowcasting_dataset.data_sources.pv.pv_data_source.drop_pv_systems_which_produce_overnight expected power values to be normalised. But, when drop_pv_systems_which_produce_overnight is called, it is given raw un-normalised power values.

@JackKelly JackKelly added the bug Something isn't working label May 17, 2022
@JackKelly JackKelly moved this to Todo in Nowcasting May 17, 2022
@JackKelly JackKelly changed the title Bug: drop_pv_systems_which_produce_overnight expected normalised data, but it doesn't receive normalised data. Bug: drop_pv_systems_which_produce_overnight expects normalised data, but it doesn't receive normalised data. May 17, 2022
JackKelly added a commit that referenced this issue May 17, 2022
@peterdudfield
Copy link
Contributor

Why does it matter if its normalised or not?

@JackKelly
Copy link
Member Author

JackKelly commented Jun 30, 2022

Here's the code (nowcasting_dataset/data_sources/pv/pv_data_source.py):

image

Note that the threshold (defined on line 546) is a "normalised" threshold. If the PV power isn't normalised, then the threshold is way too low to have much effect!

I'm pretty confident this is a "real" bug 🙂

@peterdudfield
Copy link
Contributor

ah i forgot about the threshold - thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
No open projects
Status: Todo
Development

No branches or pull requests

2 participants