-
Notifications
You must be signed in to change notification settings - Fork 2
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
Central #58
Central #58
Changes from all commits
643e60b
44f616e
8924724
bd0f0a7
e4a4413
fe7596b
212c715
080a06e
c367bdd
fbb8c23
1a88619
c71d646
9a9ab35
4b5b4b9
1e4e7ec
b78d67e
98ec6ae
74f9d7a
4f739e0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,21 @@ | ||
version: "3.0" | ||
version: "3.1" | ||
|
||
rules: | ||
- rule: Ask the user to rephrase whenever they send a message with low NLU confidence | ||
- rule: Give user central mode options when they send a message with low NLU confidence | ||
steps: | ||
- intent: nlu_fallback | ||
- action: utter_default | ||
- action: utter_central_mode_options | ||
- action: action_end_dialog | ||
|
||
# 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. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 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. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @wbaccinelli Ah good point. I will adapt the action_end_dialog action. |
||
steps: | ||
- intent: urgent | ||
- action: utter_refer | ||
- action: utter_central_mode_options | ||
- action: action_end_dialog | ||
Comment on lines
+15
to
+18
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 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? |
||
|
||
- rule: Ask for foreseen smoking HRS whenever this is externally triggered (by scheduler) | ||
steps: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
# Stories for rescheduling dialog | ||
version: "3.0" | ||
version: "3.1" | ||
|
||
stories: | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,13 @@ | ||
intents: | ||
- urgent | ||
|
||
actions: | ||
- action_end_dialog | ||
- mark_dialog_as_completed | ||
- action_handle_urgent_intent | ||
|
||
|
||
responses: | ||
# To refer a user to a human | ||
utter_refer: | ||
- text: "Volgens mij heb je een probleem waar een mens je beter mee kan helpen. Ik raad je aan om contact op te nemen met iemand die je vertrouwt, zoals je huisarts." |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
intents: | ||
- get_first_aid_kit | ||
- CENTRAL_get_first_aid_kit | ||
slots: | ||
### First aid kit | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,6 @@ | ||
git+https://github.com/PerfectFit-project/niceday_client | ||
protobuf==3.20.1 # temporary fix | ||
rasa==3.1.0 # NB! when updating, make sure to also update: | ||
rasa==3.2.8 # NB! when updating, make sure to also update: | ||
# * rasa base image in actions/Dockerfile, | ||
# * version number in nlu.yml, story files, rules.yml, | ||
# * format of stories, domain.yml etc. if necessary |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nelealbers I think this needs to be changed, but everything is working now! Nice work!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nelealbers ok, perfect! I will merge the branch then.