MycroftAI / skill-alarm

Mycroft AI official Alarm Skill - Set single and recurring alarms, with a choice of alarm sounds
https://mycroft.ai/skills
Apache License 2.0
14 stars 32 forks source link

Alarm skill #75

Open jerryar opened 4 years ago

jerryar commented 4 years ago

When attempting to set an alarm today, using Picroft, for 1:00 PM today (while the current time was 11:59 AM local), Mycroft set the alarm for tomorrow at that time. When asking Mycroft to cancel the alarm, the response was that the request was not understood. When asking Mycroft "Are any alarms set?" the response was that the request was not understood.

krisgesling commented 4 years ago

Hey Jerry, thanks for reporting this and helping us improve Mycroft.

I've been trying to replicate it but couldn't get the same behaviour. It was correctly setting the alarm in 61 minutes time.

Can you check if your running the latest version of the Skill? It should update automatically, unless it believes you are editing the Skill.

The easiest way might be to uninstall and reinstall the Skill. You can do this by saying: "Hey Mycroft, uninstall alarm" Confirm and allow it to uninstall, then: "Hey Mycroft, install alarm" Confirm and all it to install.

jerryar commented 4 years ago

Thanks for the follow up Kris. I un-installed and re-installed the alarm on your suggestion.

At about 11:35 AM today I requested Mycroft to “Set alarm for one PM”. The reply was “I have set alarm for 9 AM tomorrow 22 hours from now”. Does that make any sense to you? I confirmed that Mycroft recognizes my location at Fredericton NB in Canada, which should account for the time zone I hope.

I have tried several more times annunciating carefully but am getting the same results every time.

Then I tried typing the request in terminal mode and got the same response (snip attached)

Alarm.JPG

Cheers,

Jerry

From: Kris Gesling [mailto:notifications@github.com] Sent: Tuesday, January 28, 2020 10:38 AM To: MycroftAI/skill-alarm Cc: jerryar; Author Subject: Re: [MycroftAI/skill-alarm] Alarm skill (#75)

Hey Jerry, thanks for reporting this and helping us improve Mycroft.

I've been trying to replicate it but couldn't get the same behaviour. It was correctly setting the alarm in 61 minutes time.

Can you check if your running the latest version of the Skill? It should update automatically, unless it believes you are editing the Skill.

The easiest way might be to uninstall and reinstall the Skill. You can do this by saying: "Hey Mycroft, uninstall alarm" Confirm and allow it to uninstall, then: "Hey Mycroft, install alarm" Confirm and all it to install.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/MycroftAI/skill-alarm/issues/75?email_source=notifications&email_token=AISEVBSCNA6D4HMW66AYLWTRAA7NDA5CNFSM4KLX2ACKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKDQZGQ#issuecomment-579275930 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AISEVBQMJPQNVLMRRVV6GTDRAA7NDANCNFSM4KLX2ACA .https://github.com/notifications/beacon/AISEVBRRMEHUEU3QBILEIB3RAA7NDA5CNFSM4KLX2ACKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKDQZGQ.gif

jerryar commented 4 years ago

Alarm