Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error in debug flow #8

Open
Ostrochibi opened this issue Jun 5, 2020 · 5 comments
Open

Error in debug flow #8

Ostrochibi opened this issue Jun 5, 2020 · 5 comments

Comments

@Ostrochibi
Copy link

Hi,

Nice job for this skill, very usefull. Thank you

When you trigger the converse sequence in debug flow, mycroft says skill.error, is that normal?

I try to understant how converse works but you have made node example. Can you provide an example of a simple conversation pls, that would be very usefull.

So far i understand, all solution i found seems to huge to be correct. It's seems like you have a the node "filter" in the flow "bus" and you use a flow variable but the two possible way you add to this node point at the same place. I am lost :)

Thank you again.

@JarbasAl
Copy link
Collaborator

JarbasAl commented Jun 5, 2020

in this skill context, converse just means "before mycroft intents"

for example the included weather intent has a collision with mycroft weather skill, with converse on node red will be executed, with converse off mycroft's skill will be executed

skill.error is not normal, i will try to replicate on my side, if you could provide skill logs that would be great

@JarbasAl
Copy link
Collaborator

JarbasAl commented Jun 5, 2020

the flow merges converse/fallback into a single parsing step, you could however have different intents for each stage if you want to split both of those

in general this will not be necessary, i decided it was best to have a single place to handle intents and a simple mechanism to control converse/fallback

@Ostrochibi
Copy link
Author

Ok, i gonna give you an example to add to your repo if you want when i finish about how i map the converse things.
Can you tell me more about the skill.error provide by mycroft when you try the debug flow pls?

@keiner99
Copy link

I changed to this skill, works like a charm and needs fewer dependencies: https://github.com/erikkt/myqtt

@JarbasAl
Copy link
Collaborator

@keiner99 please dont advertise other skills in every issue... I don't mind you linking to other skills, but i mind spamming different issues with the same thing over and over.

that needs fewer dependencies because it does totally different things

a quick summary:

  • my skill is not about lang support, home assistant, or mqtt
  • my skill is quite old and needs a big version bump, as you can see from issues it does not work in latest mycroft (the install doesn't, the skill itself does if you handle that manually)
  • my skill uses the hivemind and isolates the messagebus, with the one you are advertising you are vulnerable , https://github.com/Nhoya/MycroftAI-RCE
  • DO NOT EXPOSE YOUR MESSAGEBUS
  • DO NOT EXPOSE YOUR MESSAGEBUS SPECIALLY WITHOUT AUTHENTICATION

note: the skill you linked does not provide a license either

I will try to find some time to give a cleanup to my skill

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants