Skip to content

Commit

Permalink
fix: runtime requirements reading
Browse files Browse the repository at this point in the history
  • Loading branch information
JarbasAl committed Dec 9, 2024
1 parent e6e333f commit 582d485
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions ovos_core/skill_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -303,7 +303,8 @@ def load_plugin_skills(self, network=None, internet=None):
LOG.info(f"Consider uninstalling {skill_id} instead of blacklisting it")
continue
if skill_id not in self.plugin_skills and skill_id not in loaded_skill_ids:
skill_loader = self._get_plugin_skill_loader(skill_id, init_bus=False)
skill_loader = self._get_plugin_skill_loader(skill_id, init_bus=False,
skill_class=plug)
requirements = skill_loader.runtime_requirements
if not network and requirements.network_before_load:
continue
Expand All @@ -330,7 +331,7 @@ def _get_internal_skill_bus(self):
bus = self.bus
return bus

def _get_plugin_skill_loader(self, skill_id, init_bus=True):
def _get_plugin_skill_loader(self, skill_id, init_bus=True, skill_class=None):
"""Get a plugin skill loader.
Args:
Expand All @@ -343,7 +344,10 @@ def _get_plugin_skill_loader(self, skill_id, init_bus=True):
bus = None
if init_bus:
bus = self._get_internal_skill_bus()
return PluginSkillLoader(bus, skill_id)
loader = PluginSkillLoader(bus, skill_id)
if skill_class:
loader.skill_class = skill_class
return loader

def _load_plugin_skill(self, skill_id, skill_plugin):
"""Load a plugin skill.
Expand All @@ -355,7 +359,7 @@ def _load_plugin_skill(self, skill_id, skill_plugin):
Returns:
PluginSkillLoader: Loaded plugin skill loader instance if successful, None otherwise.
"""
skill_loader = self._get_plugin_skill_loader(skill_id)
skill_loader = self._get_plugin_skill_loader(skill_id, skill_class=skill_plugin)
try:
load_status = skill_loader.load(skill_plugin)
except Exception:
Expand Down

0 comments on commit 582d485

Please sign in to comment.