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.

Sign up for GitHub

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

Jump to bottom

Eng 815 updated tests #13

Closed
wants to merge 36 commits into from
Closed

Eng 815 updated tests #13

wants to merge 36 commits into from

Conversation

twerkmeister
Copy link
Collaborator

Description

  • updated tests with latest changes from main
  • fixed a few bugs in the command annotations around None/null and ordering of clarification options

twerkmeister and others added 30 commits February 1, 2024 15:51
@twerkmeister
fixed disambiguation and negation tests
7345cc9
@twerkmeister
fixed utterance name
2b8d8f3
@twerkmeister
test fixes
e45f65b
@twerkmeister
removed cancel confirmation for non transfer money flow
33d8da2
@twerkmeister
removed second time string from test
17d09a0
@twerkmeister
reinstated misunderstood test
1b91278
@twerkmeister
moved back two tests to passing where they originally where
78c8891
@twerkmeister
using intentless instead of enterprise search for testability
e3ff5ba
@twerkmeister
reinstated removed test
5eba7a4
@OgnjenFrancuski
Add info about setting up RASA_DUCKLING_HTTP_URL environment variable. (
2627af5 
#8)
@twerkmeister
Merge branch 'main' into ENG-766-test-fixes-multi-prompting-spike
78cca6d
@twerkmeister
readded slot sets for restaurant time and date but without specified …
ca3319a 
…values
@twerkmeister
added checks for setting to none
5a880f5
@twerkmeister
moved knowledge tests to passing bc using intentless makes them possible
5374401
@twerkmeister
added duckling url secret to env
7b004db
@varunshankar
add prompts
1ab5ef7
@twerkmeister
moved slot sets to the right user message
a94feba
@twerkmeister
setting temp and top p explicitly
76eb79a
@varunshankar
add prompts
2bd8e9c
@varunshankar
fix prompts
720daa1
@varunshankar
fix review comments
2dd63b3
@varunshankar
update readme
09953ae
@varunshankar
Update PROMPT_README.md
eeadf77
@varunshankar
Merge pull request #9 from RasaHQ/ENG-814-add-additional-prompts
8081121 
Add alternative prompts - [ENG 814]
@OgnjenFrancuski
Update rasa and rasa-plus to 3.7.8 (#11)
290a6a9
@twerkmeister
Merge branch 'main' into ENG-766-test-fixes-multi-prompting-spike
dd4b16c
@twerkmeister
added flaky test category, ci step, and PR template
1244eff
@twerkmeister
moved another test to flaky
4524b4c
@twerkmeister
passing step for flaky tests
bceb133
@twerkmeister
moved tests that are actually passing out of failing category
3c8c38e
@twerkmeister twerkmeister deleted the ENG-815-updated-tests branch March 11, 2024 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@twerkmeister @OgnjenFrancuski @varunshankar