openclimatefix / uk-pvnet-app

Application for running PVNet in production
MIT License
1 stars 4 forks source link

Make post sunset filter more lenient #54

Open dfulu opened 8 months ago

dfulu commented 8 months ago

Currently we set the prediction for a timestep to zero if we calculate the sun to be below the horizon at that time. However, the timestamp we use for any period is at the end of that period. E.g. the timestamp for 18:00 actually covers the PV output from 17:30 to 18:00. Although the sun may have set at the end of the period, it may still have been up for part of it. We should loosen up the night time filter to allow for this

Harshcreator commented 7 months ago

Title: Loosen nighttime filter for PV output predictions

Description: Hi,

I hope this finds you well. I've addressed the issue regarding the nighttime filter for PV output predictions in this pull request. Currently, the code was overly restrictive in setting predictions to zero if the sun was calculated to be below the horizon at the end of each period. However, I've made adjustments to the code to account for the fact that the timestamp used for any period actually covers the PV output from the halfway point of the previous period to the end of the current period.

Changes Made:

Testing:

Additional Notes:

I'd appreciate your feedback on these changes. Please let me know if there are any further adjustments or tests you'd like me to perform.

Best regards, Harsh Singh