Comments (10)
Replicated under 18.2.13 on Ubuntu 18.04 running stable
branch from master
02:21:43.495 - SKILLS - DEBUG - {"type": "mycroft.skill.handler.start", "data": {"name": "TimerSkill.handle_start_timer"}, "context": {}}
02:21:43.496 - mycroft.skills.core:wrapper:655 - ERROR - An error occurred while processing a request in Timer Skill
Traceback (most recent call last):
File "/home/kathyreid/mycroft-core/mycroft/util/lang/format_en.py", line 292, in pronounce_number_en
last = number_names[int(_num[2:4])]
ValueError: invalid literal for int() with base 10: '.0'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/kathyreid/mycroft-core/mycroft/skills/core.py", line 646, in wrapper
handler(message)
File "/opt/mycroft/skills/mycroft-timer.mycroftai/__init__.py", line 171, in handle_start_timer
timer_name = nice_duration(self, secs, lang=self.lang)
File "/opt/mycroft/skills/mycroft-timer.mycroftai/__init__.py", line 697, in nice_duration
out += pronounce_number(minutes, lang) + " " + self.translate(
File "/home/kathyreid/mycroft-core/mycroft/util/format.py", line 276, in pronounce_number
scientific=scientific)
File "/home/kathyreid/mycroft-core/mycroft/util/lang/format_en.py", line 300, in pronounce_number_en
print('Exception in pronounce_number_en' + e)
TypeError: must be str, not ValueError
02:21:43.497 - SKILLS - DEBUG - {"type": "speak", "data": {"utterance": "An error occurred while processing a request in Timer Skill", "expect_response": false}, "context": {}}
from mycroft-timer.
Bug confirmed, flagging internally
from mycroft-timer.
I tried adding a cast to float()
before int()
on lines 290 and 292 but this led to
I'm setting a timer for ten zero minutes
- ie. it appended the 'zero` word.
from mycroft-timer.
Is this still an issue on the 18.08 branches? It seems to be working for me
from mycroft-timer.
from mycroft-timer.
Yeah, ideally I wouldn't ever have a regression, but going back to fix bugs when an upcoming release (in 48 hours) takes care of it could consume all our time.
from mycroft-timer.
from mycroft-timer.
We are still working on the testing frameworks. This got broken by a change on mycroft-core, not a change in the skill itself. This really is a complex ecosystem, but we are getting the test bits in place to catch more and more to catch things like this.
from mycroft-timer.
from mycroft-timer.
No worries at all, I don't feel too raked over the coals. ;-)
There is a reason we are still calling this "beta". Even stable is still in flux. We are racing to get things in shape to really be in shape for the general consumer with the 19.02 release -- which sounds like a long time, but there is a lot to be done.
One of the paradoxes of software development is that I can iterate rapidly on my own, but the moment you have a user base to support you suddenly have to think about upgrade paths, etc. And with Mycroft we are also adding in more complexity due to the skill system which pulls in the work of others -- I can't make a change and go in to change their stuff too, so I am restricted in what changes I can make.
Not complaining or justifying, it is just the way it is. I actually enjoy this kind of challenge 99% of the time.
from mycroft-timer.
Related Issues (17)
- Only trigger wakeword event handler if timer is active/expired HOT 1
- Expiry of a named timer should always announce the name
- Weird visuals after timer finishes HOT 1
- Picroft hanging up and overheating HOT 3
- Escalation of unacknowledged timer expiration enhancement
- Show timer on faceplate HOT 4
- mycroft-core keep saying "the timer <some time> minutes is up" even after restarted HOT 1
- Create 3 minute timer HOT 2
- Accept "stop" without the wake word to stop a beeping timer
- Add Stopwatch functionality HOT 3
- 60 minutes HOT 2
- timer set for null minutes. HOT 10
- [WIP] Adding interval feature to timer HOT 1
- Possible bug in cancel handler HOT 1
- Add configurable timer beep HOT 1
- An Error occurred while processing a request in Timer Skill HOT 7
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from mycroft-timer.