Wissenschaftsstadt Darmstadt + Darmstadt-Dieburg hinzufügen

[LearningLearning9]

Hallo Thomas & Co,

hier die Daten für die Stadt Darmstadt.

Falls etwas nicht passt könnt ihr gerne die Sachen passend editieren.

Grüße
Markus

[LearningLearning9]

ich erkenne auf die schnelle nicht, was der Grund für die fehlgeschlagenen checks ist.
Da du/ihr das wahrscheinlich direkt wisst investiere ich da mal keine weitere Zeit :)

[silberzwiebel]

Hi Markus,

danke! Es waren noch ein paar Fehler in den Daten (u.a. Jahreszahlen stimmten nicht, CO2 in Kilotonnen, nicht Tonnen). Habe das direkt bei Dir geändert. Du müsstest jetzt noch lokal bei Dir python create_markdown.py darmstadt ausführen und das Ergebnis davon pushen. Falls das bei Dir nicht klappen sollte, sag Bescheid.

[silberzwiebel]

Danke fürs Update! Preview ist hier: https://deploy-preview-151--klimawatch.netlify.app/kommunen/darmstadt/

Wenn alles soweit in Ordnung ist, dann sag Bescheid und ich stelle es mit einem Merge online.

[silberzwiebel]

Ping @LearningLearning9

[LearningLearning9]

Hey,
danke für den Reminder.
Sieht soweit alles gut aus.

Ready to merge von meiner Seite.

[LearningLearning9]

habe noch Daten für Darmstadt-Dieburg hinzugefügt.
Leider nicht so viele vorhanden wie bei Darmstadt.

[LearningLearning9]

Bei der Durchsicht ist grade aufgefallen, dass die datei data/darmstadt-dieburg.csv vom .gitignore ignoriert wurde.

.. habe sie nun per git add -f hinzugefügt.
Würde mich über eine Erklärung freuen ^^

[LearningLearning9]

ping @silberzwiebel

[silberzwiebel]

Warum die Datei für Darmstadt-Dieburg nicht mitgekommen ist, weiß ich leider nicht. Erste Vermutung: Der Bindestrich?

Sonst folgende Dinge bitte noch berücksichtigen:

Es ist aber tatsächlich noch eine weitere Datei nicht mitgekommen, und zwar die Datei hugo/data/you_draw_it_darmstadt-dieburg.html. Deshalb gibt es dort noch keinen Plot zum selber Zeichnen. Der zweite Plot bei Darmstadt-Dieburg sieht aber tatsächlich auch etwas kaputt aus, vermutlich weil die Daten so alt sind. Deshalb rechnet das Skript wohl aus, dass bereits 2017 das Paris-Budget aufgebraucht wurde ... Kann ich trotzdem mergen, könnte aber Fragen aufwerfen. Vielleicht gibt es ja doch noch aktuellere Daten?

Dann noch eine weitere Sache: Die E-Mailadressen sollen in der .md-Datei tatsächlich falschrum drinstehen. Das ist ein rudimentärer Spamschutz, die werden wieder korrekt umgedreht (sind also jetzt falsch, siehe Preview: https://deploy-preview-151--klimawatch.netlify.app/ ).