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

[JackKelly]

trafficstars

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.

[peterdudfield]

Why does it matter if its normalised or not?

[JackKelly]

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

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 :slightly_smiling_face:

[peterdudfield]

ah i forgot about the threshold - thanks