Skip to content

Commit

Permalink
rename padatious:train to mycroft.skills.train to keep things intent …
Browse files Browse the repository at this point in the history
…engine agnostic
  • Loading branch information
JarbasAl committed Dec 9, 2024
1 parent c8e8195 commit c74503d
Showing 1 changed file with 6 additions and 8 deletions.
14 changes: 6 additions & 8 deletions ovos_core/skill_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
import os
from os.path import basename
from threading import Thread, Event, Lock

from time import monotonic

from ovos_bus_client.apis.enclosure import EnclosureAPI
Expand Down Expand Up @@ -118,8 +117,9 @@ def __init__(self, bus, watchdog=None, alive_hook=on_alive, started_hook=on_star
self._logged_skill_warnings = list()
self._detected_installed_skills = bool(find_skill_plugins())
if not self._detected_installed_skills:
LOG.warning("No installed skills detected! if you are running skills in standalone mode ignore this warning,"
" otherwise you probably want to install skills first!")
LOG.warning(
"No installed skills detected! if you are running skills in standalone mode ignore this warning,"
" otherwise you probably want to install skills first!")

self.config = Configuration()

Expand Down Expand Up @@ -513,17 +513,15 @@ def _load_new_skills(self, network=None, internet=None, gui=None):
if loaded_new:
LOG.info("Requesting padatious intent training")
try:
response = self.bus.wait_for_response(Message("padatious:train"),
"mycroft.skills.trained",
timeout=60) # 60 second timeout
response = self.bus.wait_for_response(Message("mycroft.skills.train"),
"mycroft.skills.trained",
timeout=60) # 60 second timeout
if not response:
LOG.error("Padatious training timed out")
elif response.data.get('error'):
LOG.error(f"Padatious training failed: {response.data['error']}")
except Exception as e:
LOG.exception(f"Error during padatious training: {e}")
else:
LOG.debug("Nothing new to train")

def _get_skill_loader(self, skill_directory, init_bus=True):
"""Get a skill loader instance.
Expand Down

0 comments on commit c74503d

Please sign in to comment.