Nonetype Error: Can´t get weather on German PiCroft Installation about skill-weather HOT 4 OPEN

Hey there, since I also experienced this bug on my German Picroft, I had another look at it.
I was able to trace it back to the function __populate_report, as can already be seen from the log output. This function invokes __extract_datetime("today"), and thereby the mycroft-core function, which returns None, as long as the default language is set to anything else than English (because only the English parser knows the word "today").

If the language is passed together with "today" (lang='en'), the function will return the correct date and time, and the skill works as expected.
Therefore, I changed the function call in __populate_report, and a similar call in __populate_current that raised an error as well.

Although I'm very new to Mycroft and Github, I will try to open a pull request.

from skill-weather.

Hey there, thanks for reporting this bug.

This Skill will soon be largely rewritten but if anyone wants to patch this in the interim, we'll gladly accept a PR.

from skill-weather.

@krisgesling ok thank you for your answer

from skill-weather.

Thanks for digging into this further @junivex

Let us know if you run into any blockers dealing with Github

from skill-weather.