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

skill.error while asking for the weather #202

Open
0xsharkboy opened this issue May 17, 2022 · 12 comments
Open

skill.error while asking for the weather #202

0xsharkboy opened this issue May 17, 2022 · 12 comments
Labels

Comments

@0xsharkboy
Copy link

Hey, i'm on the picroft version and it returns a skill.error when i'm asking for the weather:

17:52:10.347 | INFO | 784 | WeatherSkill | Speaking dialog: current-weather-local 17:52:10.364 | ERROR | 784 | mycroft.skills.mycroft_skill.mycroft_skill:on_error:923 | skill.error Traceback (most recent call last): File "/home/pi/mycroft-core/mycroft/skills/mycroft_skill/event_container.py", line 73, in wrapper handler(message) File "/opt/mycroft/skills/mycroft-weather.mycroftai/__init__.py", line 122, in handle_current_weather self._report_current_weather(message) File "/opt/mycroft/skills/mycroft-weather.mycroftai/__init__.py", line 667, in _report_current_weather self._speak_weather(dialog) File "/opt/mycroft/skills/mycroft-weather.mycroftai/__init__.py", line 1186, in _speak_weather self.speak_dialog(dialog.name, dialog.data, wait=True) File "/home/pi/mycroft-core/mycroft/skills/mycroft_skill/mycroft_skill.py", line 1227, in speak_dialog self.dialog_renderer.render(key, data), File "/home/pi/mycroft-core/mycroft/dialog/dialog.py", line 110, in render line = line.format(**context) KeyError: 'low_temperature'

If someone can help me it would be very kind :D
I need to specify that I changed the laguage to French In the Mycroft's config file

@0xsharkboy 0xsharkboy added the bug label May 17, 2022
@0xsharkboy
Copy link
Author

image

@forslund
Copy link
Collaborator

I think this would be fixed by PR #118 or if the translate job is run by the mycroft team (translations seems to be updated on the translate server). @krisgesling might be able to say if it would be best to use the MR or if it's time to run the automated job.

@0xsharkboy
Copy link
Author

Okay thanks :D

@krisgesling
Copy link
Contributor

Hey there, running the translation export now so this should be fixed shortly.

@krisgesling
Copy link
Contributor

I think the translation scripts are having trouble with the new directory structure.

So would accept a pull request in the meantime if you wanted to.

@0xsharkboy
Copy link
Author

Hey, I made my own traduction. Does it interest you ?

@krisgesling
Copy link
Contributor

Yeah that would be great if you wanted to do a PR :)

@0xsharkboy
Copy link
Author

Okay, I juste don't know how to upload this

@krisgesling
Copy link
Contributor

Hey if you're new to working with git there are some good guides from Github that will help you get started.

If you've made the changes in the Skill (and this is the only change you've made), you can fork the repo at the top of this page, then you'd want to run something like:

cd /opt/mycroft/skills/mycroft-weather.mycroftai/
git remote add sharkboy https://github.com/SharkBoy69-CS/skill-weather/
git checkout -b bugfix/french-translations
git commit -am "Update French translations"
git push sharkboy bugfix/french-translations

@0xsharkboy
Copy link
Author

Hey, thanks ! Is it that ? https://github.com/SharkBoy69-CS/skill-weather

@valvin1
Copy link

valvin1 commented Jul 3, 2022

hello, i did the same issue and discover that the fix provided the 2nd of june was not yet published on a french install. I had to:

cd /opt/mycroft/skills/mycroft-weather.mycroftai/
git pull
Mise à jour 481e6d4..9acb334

why the command which update skills didn't update automatically ?

@apennamen
Copy link

Hello,

Starting with a fresh install of picroft, and switching to french, I stumbled upon the same issue.
The weather skill was pointing to the 20.08 branch instead of the 21.02.

After checking out the last branch, the error vanished.

Regards,
AP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants