Central #58
Central #58
Conversation
…al intents CENTRAL...
|
@wbaccinelli From what I can see, the failed Python CI test has nothing to do with what I have done. For the Rasa CI, does it fail because we have now split up the files? |
|
@nelealbers I will take a look at the python CI issues. The Rasa CI has been already fixed here to take into account the file splitting. The log says that the intents 'urgent' and 'CENTRAL_get_first_aid_kit' are not part of the domain. Can you check this? |
|
@wbaccinelli Thanks, fixed! |
|
|
||
| # Only applies at start of conversation | ||
| - rule: Process when people talk about urgent things that we cannot handle | ||
| conversation_start: true |
There was a problem hiding this comment.
This is a nice solution to avoid this to be triggered during the dialogs, but the action_end_dialog does not start a new conversation, but resets the slots through the default action_restart. So, also the action_end_dialog should be modified to make it start a new conversation maybe using also action_session_start.
Another possible solution would be to use a rule with condition that checks that the current_intervention_component slot is not set.
There was a problem hiding this comment.
@wbaccinelli Ah good point. I will adapt the action_end_dialog action.
| - intent: urgent | ||
| - action: utter_refer | ||
| - action: utter_central_mode_options | ||
| - action: action_end_dialog |
There was a problem hiding this comment.
I tried this, but the confidence level of confidence is too low (even if I write precisely one of the examples). I think we have to pay a lot of attention on how we train the model. Maybe we should have different intents for different words?
…al-coach-rasa into central merging walter's fix
|
@wbaccinelli I now got things to work. There was a bug that sometimes made things break that was fixed in 3.2.8: https://rasa.com/docs/rasa/changelog. So I moved everything to 3.2.8. As to the rasa restart vs. new conversation start, the restart removes all slots and conversation history, while the new conversation start only removes the conversation history (i.e., not the slots). I think it is the restart behavior that we want or not? Let me know if I should merge this now. |
|
SonarCloud Quality Gate failed. |
| FROM rasa/rasa-sdk:3.1.1 | ||
| FROM rasa/rasa-sdk:3.2.1 |
There was a problem hiding this comment.
| FROM rasa/rasa-sdk:3.1.1 | |
| FROM rasa/rasa-sdk:3.2.1 | |
| FROM rasa/rasa-sdk:3.2.8 |
There was a problem hiding this comment.
@nelealbers I think this needs to be changed, but everything is working now! Nice work!
There was a problem hiding this comment.
@wbaccinelli I thought so at first too, but it seems that rasa-sdk has different non-matching version numbers: https://rasa.com/docs/action-server/rasa-sdk-changelog/. Their newest version is 3.2.1 according to their changelog.
There was a problem hiding this comment.
@nelealbers ok, perfect! I will merge the branch then.
Fixes PerfectFit-project/virtual-coach-issues#266.
Changes: