-
Notifications
You must be signed in to change notification settings - Fork 925
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
Add solar production data for Croatia #1987
Conversation
36c6fa9
to
4f20116
Compare
@martincollignon could you please review this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Bonus points for adding tests. Minor comments added.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @OlgaCh thanks for the PR, can you post the output from the HOPS parser?
Updated PR to address requested changes. Also noticed that in some cases data file with the solar data estimation may be not available for the current date since being updated with the delay. |
@corradio @systemcatch kindly check updated pr. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good @OlgaCh!
Closes #1966
Updates HOPS parser with the additional data source with the estimated power production by solar panels: https://www.hrote.hr/planning-electricity-production-for-the-eco-balance-group