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.

Sign up for GitHub

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

Jump to bottom

Wissenschaftsstadt Darmstadt + Darmstadt-Dieburg hinzufügen #151

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Wissenschaftsstadt Darmstadt + Darmstadt-Dieburg hinzufügen #151

wants to merge 14 commits into from

Conversation

LearningLearning9
Copy link

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

Markus Heidemann added 3 commits December 9, 2021 17:29
@LearningLearning9
Copy link
Author

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
Update darmstadt.csv
86ac979 
- Emissionen in Kilotonnen
- last_emissions keyword fehlte
- nur eine Einwohnerzahl wird benutzt
@silberzwiebel
add darmstadt to meta
afa8524 
das war schon richtig hier
@silberzwiebel
Copy link
Member

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.

generate markdown
600130e 
done with
`python create_markdown.py`
@silberzwiebel
Copy link
Member

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
Copy link
Member

Ping @LearningLearning9

@LearningLearning9
Copy link
Author

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

Ready to merge von meiner Seite.

@LearningLearning9 LearningLearning9 changed the title Wissenschaftsstadt Darmstadt hinzufügen Wissenschaftsstadt Darmstadt + Darmstadt-Dieburg hinzufügen Mar 19, 2022
@LearningLearning9
Copy link
Author

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

Markus Heidemann added 2 commits March 19, 2022 10:52
fix emails + remove pdf
415cf22
add data of darmstadt-dieburg
dcb54dd 
* somehow gitignore does filter this file
* added with git add -f
@LearningLearning9
Copy link
Author

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
Copy link
Author

ping @silberzwiebel

@silberzwiebel
Copy link
Member

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/ ).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@LearningLearning9 @silberzwiebel