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

Bugfixes weewx-wdc Version 2.3.0 #81

Closed
Daveiano opened this issue Oct 17, 2022 Discussed in #79 · 3 comments · Fixed by #82
Closed

Bugfixes weewx-wdc Version 2.3.0 #81

Daveiano opened this issue Oct 17, 2022 Discussed in #79 · 3 comments · Fixed by #82
Labels
bug Something isn't working needs tests
Milestone

Comments

@Daveiano
Copy link
Owner

Discussed in #79

Originally posted by hoetzgit October 17, 2022
@Daveiano

Hallo David,

ich bin etwas vorsichtig Issues zu generieren, weil ich eigene Fehler in meiner Konfiguration nicht ausschließen kann. daher diese Vorphase:

weewx-DWD Einbindung:

  • Bei der Vorhersagetabelle habe ich offensichtlich bei lang=de ein Umlautproblem. Habe versucht, bspw. "ö" durch ö zu ersetzen, ohne Erfolg. Auch scheinen in die de.conf z.B. Zeile 216 noch ein paar Zeichen zusätzlich mit reingerutscht zu sein.
  • Leider sind deutsche Bezeichnungen oft sehr lang. So reicht z.B. "Niederschlagswahrscheinlichkeit" bis in den Wertebereich hinein und überschreibt diesen. Gibt es eine Möglichkeit, eine Worttrennung zu erreichen?
  • Die Warnkarte "SchilderXX.jpg" ist evtl. statisch programmiert?

Benötigst Du weitere Infos, melde Dich gerne.
Siehe auch Webseite

Grüße
Henry

@Daveiano Daveiano added bug Something isn't working needs tests labels Oct 17, 2022
@Daveiano
Copy link
Owner Author

Daveiano commented Oct 17, 2022

Das sind definitiv 3 Bugs die unter der deutschen Sprache auftreten, kann ich mit meiner lokalen Installation nachvollziehen.

Das mit der Warnkarte "SchilderXX.jpg" ist mir gar nicht aufgefallen weil der Standardwert Sachsen ist, was für mich auch zutrifft. Musste ich also nie ändern, deswegen habe ich das übersehen.

Da man das händig in wget-dwd angeben muss, wird man das dann noch ein zweites mal in der skin.conf setzen müssen.

Zur Erklärung: Die Zeichen in der de.conf sind beabsichtigt. ­ ist ein Bedingter Trennstrich. Das heißt da wird nur eine Trennung vorgenommen wenn es aus Platzgründen auch nötig ist. Mehr dazu hier: https://de.wikipedia.org/wiki/Bedingter_Trennstrich. Nur sollte der Skin das auch rendern und nicht als Text darstellen, das ist der Fehler.

@Daveiano Daveiano changed the title Fragen zu weewx-wdc Version 2.3.0 Bugfixes weewx-wdc Version 2.3.0 Oct 17, 2022
@Daveiano Daveiano linked a pull request Oct 18, 2022 that will close this issue
@hoetzgit
Copy link
Contributor

@Daveiano
Vielen Dank für die Optimierungen und auch für die Erklärung zu ­ - wieder etwas gelernt.

@Daveiano
Copy link
Owner Author

@hoetzgit Gerne. Ich werde noch bis etwa Ende der Woche warten ob noch mehr Feedback zur 2.3.0 kommt und ansonsten ein Bugfix-Release veröffentlichen, da werden die Fixes von dem Issue hier und #83 enthalten sein.

@Daveiano Daveiano pinned this issue Oct 19, 2022
@Daveiano Daveiano added this to the 2.3.1 milestone Oct 20, 2022
@Daveiano Daveiano unpinned this issue Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs tests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants