rhasspy-hermes-app icon indicating copy to clipboard operation
rhasspy-hermes-app copied to clipboard

Let the app load its intents/slots/… from a file

Open koenvervloesem opened this issue 5 years ago • 3 comments
trafficstars

Let the app load its intents/slots/… from a file and re-train Rhasspy on installation/startup of the app.

See https://github.com/rhasspy/rhasspy-hermes/issues/12

koenvervloesem avatar Jun 13 '20 20:06 koenvervloesem

One thing to keep in mind with this is that it should only retrain when there was a change (either to the files or some other settings like e.g. the language) and not on each restart, since training can take very long when a lot of sentences need to be generated.

maxbachmann avatar Aug 26 '20 19:08 maxbachmann

Good point. So we should compare the locally defined intents and slots with the ones known/trained by Rhasspy then, and only trigger Rhasspy to re-train when they differ?

koenvervloesem avatar Aug 26 '20 20:08 koenvervloesem

I think it would be hard to compare them to the trained ones, since this would require generating them aswell. Reasons to retrain from the top of my head:

  • rhasspy settings change that require a retrain
  • skill files change
  • probably when the skill has a new version aswell, since following something like #13 e.g. the skill name might have changed

One more thing is that when skills retrain by sending their intents + slots we do want them to do this at the same time. It would be a complete mess when:

  • skill A sends the intents -> rhasspy starts to retrain
  • skill B sends the intents -> rhasspy has to retrain again

I am currently not sure how to implement this in an optimal way.

maxbachmann avatar Aug 26 '20 20:08 maxbachmann