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

Änderungen des Skriptes für die Regionallogos #1

Merged
merged 2 commits into from
Sep 18, 2019

Conversation

g-voelker
Copy link
Contributor

Ich habe zwei Änderungen vorgenommen:

  1. Ersetzen des plotting scripts mit den Warming Stripes aus dem bundesweiten Logo
  2. Einfügen eines weitern Aufrufs von Inkscape um den Text im PDF in Pfade umzuwandeln

Jetzt müssten die erzeugten Logos von den Farben passen und auf alle Maschinen portabel sein.

@ccauet ccauet requested a review from maxnoe September 10, 2019 07:29
@ccauet
Copy link
Member

ccauet commented Sep 10, 2019

Hi @g-voelker, ich würde stark bevorzugen auf Basis eines abgestimmten Datensatzes, eines Betrachtungszeitraums und einer Farbpalette zu arbeiten anstatt eine statische Datei einzubinden.

Vor allem wichtig wäre mir aber einmal zu klären, was das überhaupt für ein Plot ist den wir im bundesweiten Logo nutzen. Welche Originaldaten, Transformationen(?), Zeitraum, Farbpalette liegen da zugrunde?

Ob dann in jedem Aufruf geplottet werden muss oder nur einmal, ist dann vermutlich auch wieder zweitrangig. Aber es wäre doch cool ein Skript zu haben mit dem wir die Stripes reproduzierbar erstellen können.

@g-voelker
Copy link
Contributor Author

Ich stimme zu, dass ein erstellter Plot sehr viel eleganter wäre. Die statische Datei ist auch quasi nur die "quick and dirty" Variante. Vorschlag: wir könnten die ganze Sache branchen und einen branch mit der jetzigen statischen Datei aufstellen, denn das läuft jetzt sofort. Dann ist der Zeitdruck aus der eleganten Lösung auch raus.

@ccauet
Copy link
Member

ccauet commented Sep 10, 2019

Ja, das wäre ne Option. Die Logos jetzt schnell verfügbar machen, ist sicher ein sinnvolles Anliegen. Zu der Frage wie genau der Warming Stripes Plot zustande kommt, dann weiter in RocketChat?

@g-voelker
Copy link
Contributor Author

Das klingt gut. Muss ich die Änderungen jetzt in einen neuen Branch schieben, oder kannst du so mit dem pull request verfahren?

@ccauet
Copy link
Member

ccauet commented Sep 10, 2019

Gute Frage, dass soll mal @maxnoe beantworten ;)

@maxnoe maxnoe merged commit 79df4da into s4f-dortmund:master Sep 18, 2019
maxnoe pushed a commit that referenced this pull request Jan 4, 2024
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.

3 participants