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

Allows skill to dynamically define intents or entities #3100

Closed
zeronounours opened this issue Apr 24, 2022 · 1 comment
Closed

Allows skill to dynamically define intents or entities #3100

zeronounours opened this issue Apr 24, 2022 · 1 comment

Comments

@zeronounours
Copy link

Is your feature request related to a problem? Please describe.
Some skills my require dynamic definition of padatious entities to prevent matching requests intented for other skills.
See MycroftAI/skill-homeassistant#64 and MycroftAI/skill-homeassistant#104

It can currently be done with file within resource directories, but it is not a clean solution.

Describe the solution you'd like
New methods should be implemented to dynamically load new entities or intents in padatious with any wanted name. For instance:

   def register_entity_file_as(self, filename, name):

Describe alternatives you've considered
Currently skill-homeassistant provides absolute path to a temporary file to MycroftSkill.register_entity_file(). This method may be updated to only use as name the base name of the file and not the full path.

zeronounours added a commit to zeronounours/mycroft-core that referenced this issue Apr 25, 2022
Add two new skills methods to be able to load any file as padatious
intent or entity.

==== Fixed Issues ====
Fix MycroftAI#3100

====  Tech Notes ====
NONE

====  Documentation Notes ====
NONE

==== Localization Notes ====
NONE

==== Environment Notes ====
NONE

==== Protocol Notes ====
Add two new methods to skills to register padatious intents or entities
even outside of `vocab` and `locale` directories
zeronounours added a commit to zeronounours/mycroft-core that referenced this issue Apr 29, 2022
Add two new skills methods to be able to load any file as padatious
intent or entity.

==== Fixed Issues ====
Fix MycroftAI#3100

====  Tech Notes ====
NONE

====  Documentation Notes ====
NONE

==== Localization Notes ====
NONE

==== Environment Notes ====
NONE

==== Protocol Notes ====
Add two new methods to skills to register padatious intents or entities
even outside of `vocab` and `locale` directories
@forslund
Copy link
Collaborator

forslund commented Sep 8, 2024

Closing Issue since we're archiving the repo

@forslund forslund closed this as completed Sep 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants