MycroftAI / hardware-mycroft-mark-II

Mycroft's Mark II Rpi mechanical, electrical and industrial designs
108 stars 32 forks source link

Great-weather is not avail on/in latest 06-16 #79

Open Olzeke51 opened 3 years ago

Olzeke51 commented 3 years ago

Describe the bug old weather skill can't be replaced with the 'great-weather_skill' as i understand the last merge into 20.08 & 21.02

To Reproduce ssh'd in , tried to use previous working solution (prior to merge) **this is screen copy '(.venv) mycroft@localhost:~$ mycroft-msm install https://github.com/MycroftAI/skill-weather/tree/make-weather-great-again INFO - building SkillEntry objects for all skills INFO - Downloading skill: https://github.com/MycroftAI/skill-weather/tree/make-weather-great-again INFO - Problem performing action. Restoring skill to previous state... INFO - invalidating skills cache CloneException: stderr: 'Cloning into '/tmp/tmp91szb75x'... fatal: repository 'https://github.com/MycroftAI/skill-weather/tree/make-weather-great-again/' not found '

(.venv) mycroft@localhost:~$ date
Wed Jun 16 19:00:01 UTC 2021 (.venv) mycroft@localhost:~$ mycroft-msm remove skill-weather INFO - building SkillEntry objects for all skills INFO - Best match (0.9): mycroft-weather by mycroftai INFO - Successfully removed mycroft-weather INFO - invalidating skills cache (.venv) mycroft@localhost:~$ mycroft-msm install skill-weather INFO - building SkillEntry objects for all skills INFO - Best match (0.9): mycroft-weather by mycroftai INFO - Downloading skill: https://github.com/mycroftai/skill-weather INFO - Installing system requirements... INFO - Installing requirements.txt for mycroft-weather INFO - Successfully installed mycroft-weather INFO - invalidating skills cache'

after removing the old skill , that the new one that had been merged would be installed

Log files action starts @ 18:41 UTC [06-16] mycroft-fallback-to-old_weather.tar.gz

skills_short_weather.txt

Environment (please complete the following information):

Additional context FWIW - noticed lots of errors in the full skill.log !!!

krisgesling commented 3 years ago

Hi Gary, I'm fairly sure the problem here is that MSM hadn't been updated correctly. We were using a feature branch of MSM to enable sudo-free system package installation and I forgot to update this when we updated MSM. This meant that the Mark II was stuck looking at the 20.08 Marketplace and hence would get the older version of the Weather Skill. Going to test it today, but I believe it should be resolved now.

Olzeke51 commented 3 years ago

well , several 'Restarts' after removal / install of weather skill via MSM still Old weather - latest 06-16 on screen -- ??? was there a roll-back ?? [almost remember a reboot the morning of the 17th at 02:20 EST / 06:20 UTC ] also an issue after removal/reinstall the font is no bigger that 3/32 if that much [2.4mm] you use Metric right?? restart clears the font issue Gary

On Thu, Jun 17, 2021 at 12:05 AM Kris Gesling @.***> wrote:

Hi Gary, I'm fairly sure the problem here is that MSM hadn't been updated correctly. We were using a feature branch of MSM to enable sudo-free system package installation and I forgot to update this when we updated MSM. This meant that the Mark II was stuck looking at the 20.08 Marketplace and hence would get the older version of the Weather Skill. Going to test it today, but I believe it should be resolved now.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/MycroftAI/hardware-mycroft-mark-II/issues/79#issuecomment-862904998, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGAZYR2JGDP4JE6LTCUMIK3TTFYABANCNFSM462EXPRQ .

krisgesling commented 3 years ago

There was a rollback for the latest channel on the 16th. An update had a critical error so dropped back to the prior working version. A stressful moment but was also good validation for using Pantacor - everything continued working beautifully!