Skip to content
This repository has been archived by the owner on Sep 8, 2024. It is now read-only.

Commit

Permalink
Imporove user feedback during startup
Browse files Browse the repository at this point in the history
Now the eyes go to a khaki color and "< < < LOADING < < <" displays
during the skill startup.  When Padatious finishes the first training
we now emit a "mycroft.ready" on the messagebus.

A change to the skill_mark_1 to look for "mycroft.ready" instead of
"mycroft.skills.initialized" provides a good visual change to show that
it is ready for use.
  • Loading branch information
penrods committed Feb 28, 2019
1 parent 5529acd commit 286fa63
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 1 deletion.
1 change: 1 addition & 0 deletions mycroft/res/text/en-us/message_loading.skills.dialog
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
< < < LOADING < < <
3 changes: 3 additions & 0 deletions mycroft/skills/__main__.py
Original file line number Diff line number Diff line change
Expand Up @@ -151,6 +151,9 @@ def check_connection():
bus.emit(Message("enclosure.mouth.reset"))
time.sleep(0.5)

enclosure.eyes_color(189, 183, 107) # dark khaki
enclosure.mouth_text(dialog.get("message_loading.skills"))

bus.emit(Message('mycroft.internet.connected'))
# check for pairing, if not automatically start pairing
try:
Expand Down
5 changes: 4 additions & 1 deletion mycroft/skills/padatious_service.py
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,10 @@ def train(self, message=None):
LOG.info('Training complete.')

self.finished_training_event.set()
self.finished_initial_train = True
if not self.finished_initial_train:
LOG.info("Mycroft is all loaded and ready to roll!")
self.bus.emit(Message('mycroft.ready'))
self.finished_initial_train = True

def wait_and_train(self):
if not self.finished_initial_train:
Expand Down
1 change: 1 addition & 0 deletions mycroft/skills/skill_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -412,6 +412,7 @@ def run(self):
skill_path, repr(e)))
if not has_loaded and not still_loading and len(skill_paths) > 0:
has_loaded = True
LOG.info("Skills all loaded!")
self.bus.emit(Message('mycroft.skills.initialized'))

self._unload_removed(skill_paths)
Expand Down

0 comments on commit 286fa63

Please sign in to comment.