Skip to content
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

updated the renewable energy csv #30

Merged
merged 5 commits into from
Apr 5, 2022
Merged

updated the renewable energy csv #30

merged 5 commits into from
Apr 5, 2022

Conversation

Ultramarin95
Copy link
Contributor

@Ultramarin95 Ultramarin95 commented Mar 31, 2022

This updates the current renewable list by a new one. The old one was created with excel by someone who is not working on this project anymore. The new one is created by a python script that is provided in the generator-core repo. The data differs quite a bit. I guess this is mostly due to the idea that we only list the active renewable units up to 2018. The old list contained all units until 2021. I plotted a comparison between both data.
The follwowing figure shows the sorted differences |powerNew(ags) - powerold(ags)| for all ags key in the master list on a logarithmic scale.

ComparisonOldAndNewRenawables

the y-axis enumeartes all ags keys.

Somehow the biggest difference is in the pv units (the power value for allmost all ags keys changed). My explanation for this is that we got a lot of new pv units after 2018, while the other sectors did not develop that fast.

This might also be interesting: The total values for 'DG000000' changed as follows:

pvPower Old = 52247518.47 , pv power new = 41257363.215
wind power old = 52391407.513, wind power new = 52269411.15
biomass power old = 8115504.35, biomass power new = 7709256.02
geothermal power old = 38783.8, geothermal power new = -
water power old = 3599252.11 water power new = 5508440.55

Note that i removed geothermal as a power resource as we dont use it in the generator.

@anjahoehne
Copy link
Collaborator

@curiousleo gibts du mir Bescheid, wenn du mit deinem Review durch bist? Dann kann ich die fehlenden AGS ergänzen und noch einmal alles testen :)

@curiousleo
Copy link

curiousleo commented Apr 1, 2022

Somehow the biggest difference is in the pv units (the power value for allmost all ags keys changed). My explanation for this is that we got a lot of new pv units after 2018, while the other sectors did not develop that fast.

That checks out (query):

visualization(2)
Increase in Nettonennleistung by Inbetriebnahmedatum and power source between 2019-01 and 2021-12 inclusive.

PV developed the most, followed by wind.

@Ultramarin95
Copy link
Contributor Author

Ahja! Das sollte die Änderungen ja eigentlich erklären!

@anjahoehne
Copy link
Collaborator

ok top, wir können nun die Defaultwerte für alle 11323 Einheiten (11014 Kommunen, 401 Landkreise, 16 Bundesländer plus Deutschland; Achtung 109 sind Duplikate) generieren!

@anjahoehne
Copy link
Collaborator

grafik

@anjahoehne anjahoehne merged commit 7c9cffc into main Apr 5, 2022
@anjahoehne anjahoehne deleted the replaceRenewables branch April 5, 2022 14:02
@curiousleo
Copy link

@anjahoehne wrote:

@curiousleo gibts du mir Bescheid, wenn du mit deinem Review durch bist? Dann kann ich die fehlenden AGS ergänzen und noch einmal alles testen :)

Ich hätte das wahrscheinlich sagen sollen: statt mir die Tabellen in diesem PR anzuschauen, habe ich den Code reviewed, der die Tabellen erzeugt: GermanZero-de/localzero-generator-core#200. Der ist noch nicht ganz fertig, denke ich. Deshalb ist er auch als "draft" markiert.

Es kann natürlich sein, dass es Sinn macht, diesen Zwischenstand der Daten (in diesem PR hier) schon zu mergen, um damit arbeiten zu können. Den Zeitdruck bzw. die Qualitätsansprüche kann ich nicht einschätzen. Aber ich gehe davon aus, dass der Code, der diese Tabellen generiert, sich noch ändert. Und dann werden auch die Tabellen nochmal aktualisiert werden.

Alles kein Problem, aber ich dachte, ich teile hier kurz meinen Kontext.

@anjahoehne
Copy link
Collaborator

Hey @curiousleo, ja das hatte ich zu spät gesehen. Aber danke für die Info.

Es klang eher so, als ob nur noch 'Dokumentation' fehlen würde. Ich wollte jetzt erstmal abwarten, was Leon sagt. Entweder ich reverte den Merge dann oder wir machen einfach einen neuen PR auf.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants