-
Notifications
You must be signed in to change notification settings - Fork 86
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
Validation for OSM and ENTSOE data using resource files for Austria and North Macedonia #67
Validation for OSM and ENTSOE data using resource files for Austria and North Macedonia #67
Conversation
tutorial documentation
Hello @GbotemiB! Adding a couple of comments aimed to make the code a bit easier to read.
What is your feeling about that?
Regarding the plots, I have the following comments.
|
This function descibed here is to make it easy to filter through the entsoe for just AT and MK using the geometry cordinates.
Actually, no reason in particular. i will rename the variable name 😀.
Sounds great. I shouldnt be having global variables in my functions. |
@GbotemiB amazing work and very nice presentation. I think, we need a final clean-up round, and that task will be completed. Find please the comments bellow.
# Determine if Austria (AT) is in the ENTSO-E data.
entsoe_ref["if_at"] = if_country_in_entsoe(at_country_shape, entsoe_ref) As you have explained, that is filtering rather than actually definition if the country present in the data
# Clean the Austria (AT) OSM raw data, addressing non-uniform column names and NaN values.
at_osm_raw_lines = preprocess_raw_data(at_osm_raw_lines)
# Clean the North Macedonia (MK) OSM raw data, addressing non-uniform column names and NaN values.
mk_osm_raw_lines = preprocess_raw_data(mk_osm_raw_lines) My feeling is that it would be more clear to keep both code lines together and have only one comment for both of the lines
In particular, I think that this comment should be revised:
|
@ekatef Thank you for the wonderful comments and review. |
Hi @ekatef, I have refined the conclusion in the notebook. We can start iterating over the notebook from here making it understandable for a larger audience. You can check my recent commits. |
Hey @GbotemiB and thanks for the nice conclusion draft! A couple of comments.
I'd suggest you to write the next iteration of the conclusion, giving it some structure. Making some preliminary plan may be probably helpful for that. Basing on the notebook content and the draft you have created, I'd suggest the following plan.
Then the main conclusion point would look well-reasoned ;) |
Btw, with a fresh look it feels like clarity of the notebook may be slightly improved:
|
Hi @ekatef, thanks for the review. Based on the recent comments, I have revamped the conclusion based on the suggestions. Here is the new conclusion. This analysis aimed to investigate OSM data against ENTSOE data using Austria and North Macedonia. The following process was carried out during the analysis:
The analysis showed an estimated percentage difference of 11% between osm-data and ENTSOE-data. After applying the Douglas-peucker algorithm, an estimated percentage difference of about 5% was observed in the analysis. |
Hello @GbotemiB! Thank you for working on that. My feeling is that we are converging. Comments on the overall structure of the notebook:
Regarding the Conclusion: now it reads much smoother! A couple of comments:
I think we are very close to finalise. Great work! |
Thanks for the review.
|
Hi @ekatef, I have revised the notebook based on your comments. Ready for another revision 🙃 |
Hello @GbotemiB! Perfect!! :D Great that you have found a neat way to fix numeric precision issue. And I very much like the result of you work. The only comment left is an answer to your question regarding the percentage change: I think it should be rather a relative change as compared to ENTSO reference data, as we assume them to be standard de facto. Would be also perfect to add a couple of the relative changes values to the conclusion, like: After that, I think the notebook is basically ready to merge. @pz-max do probably you have any comments or recommendations? 🙂 |
Thank you @ekatef for the review, I will make the final changes as you have suggested. |
Great work! I think, the only point left is the very final polishing of the conclusion, and the PR is ready to merge. Thanks for the amazing contribution! 😄 |
cdd8796
to
3226f69
Compare
Merged 🎉 🎉 🎉 |
Thank you for the amazing support. |
This PR contains a validation notebook that explains the discrepancy found here.
Objective
The Objective of this notebook is to validate OSM data against ENTSOE data using resource files for Austria and North Macedonia.
Results
It was confirmed that the reason for the discrepancy was because the
elec.nc
file for OSM that was compared against that of PyPSA-Eur contains voltages that are lower than 220kV that was rebased to 220kV. It is advised that comparison should be done using base_network_csv files since there is no rebase of voltage in the csv files.PS: There is an ongoing conversation about this notebook here