-
Notifications
You must be signed in to change notification settings - Fork 17
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
ENG-1293: Update rasa-calm-demo to version 3.10.1 #58
Commits on Aug 22, 2024
-
Add a new folder of updated e2e tests with assertions (#46)
* update some e2e_tests to the new assertions format * adapt 2 correction test cases * adapt all cancellations, chitchat, disambiguation * adapt test cases in invalid_path, invalid_user_inputs, negations, potential_bugs * adapt digressions, flow_guards, skip_question * adapt tests for corrections * adapt all happy_path test cases * fix error in running datetime validation in the custom actions * add generative assertion test cases * update gitignore * Add assertions for failing and flaky tests * udate rasa-pro to 3.10.0rc1 * add mlflow optional dependency, add new workflow and new make commands * fix CI deprecation warning, fix failing test case * update threshold to prevent flakiness * update button payload and test case * fix flaky passing e2e test --------- Co-authored-by: Maksim Moiseikin <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 9d37bd8 - Browse repository at this point
Copy the full SHA 9d37bd8View commit details
Commits on Aug 29, 2024
-
Add example stub custom actions (#49)
* add some updated existing tests in a separate folder, add a new CI workflow and make command * add chitchat generative assertions test case * update to rasa pro 3.10.0rc2 * set assertion_order_enabled to true in 1 test step
Configuration menu - View commit details
-
Copy full SHA for 262c711 - Browse repository at this point
Copy the full SHA 262c711View commit details -
Replace hardcoded currency strings with a regex parsing [ENG-1303]
This will fix a TypeError when we get a string like "50 dollars"
Configuration menu - View commit details
-
Copy full SHA for 60bb99e - Browse repository at this point
Copy the full SHA 60bb99eView commit details -
Merge pull request #52 from RasaHQ/feature/ENG-1303_parse_amount_of_m…
…oney Replace hardcoded currency strings with a regex parsing [ENG-1303]
Configuration menu - View commit details
-
Copy full SHA for 25866cd - Browse repository at this point
Copy the full SHA 25866cdView commit details
Commits on Sep 4, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 5f1bb49 - Browse repository at this point
Copy the full SHA 5f1bb49View commit details -
Merge pull request #53 from RasaHQ/update-to-3.10.0rc3
Update rasa-pro to 3.10.0rc3
Configuration menu - View commit details
-
Copy full SHA for 5b48d80 - Browse repository at this point
Copy the full SHA 5b48d80View commit details
Commits on Sep 9, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 48e4580 - Browse repository at this point
Copy the full SHA 48e4580View commit details
Commits on Sep 18, 2024
-
ENG-1293: Update
rasa-calm-demo
to version 3.10.1 (#57)* link to add_contact in pattern_clarification * link from check_portfolio to pattern_human_handoff * link from pattern_clarification to pattern_human_handoff * do not rephrase utter_human_handoff_not_available * ENG-1293: Added pattern_human_handoff flow * ENG-1293: Running tests with `--e2e-results`, and saving results on failure, to assist in troubleshooting * ENG-1293: Update `rasa-calm-demo` to version 3.10.1 * ENG-1293: Fix 2 failing tests Remove `utter_clarification_options_rasa` assertion in the following two failing tests, as now this step won't be reached: 1. e2e_tests_with_assertions/passing/disambiguation/user_sends_short_verb_only_message.yml::user sends short verb only message 2. e2e_tests_with_assertions/passing/negations/users_says_they_dont_want_the_former_option.yml::user says they don't want the former option * ENG-1293: fix mistake, had removed assertion from a wrong similar sounding test, corrected it now * ENG-1293: fix/update test `user_sends_short_verb_only_message`, based on recent pattern-clarification changes * ENG-1293: properly add `pattern_human_handoff` to default `pattern_clarification` * ENG-1293: Add tests to verify that link to `pattern_human_handoff` is triggered in relevant conditions On these conditions: 1. When `pattern_clarification` is triggered twice 2. Or when login fails during check-portfolio at least 3 times --------- Co-authored-by: Tanja Bunk <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 1e38976 - Browse repository at this point
Copy the full SHA 1e38976View commit details