-
Notifications
You must be signed in to change notification settings - Fork 803
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
Converting Rasa-Demo to 2.0 Format #566
Conversation
@melindaloubser1 I've converted the NLU files to 2.0 (only thing missing is the lookup table file definition in the training data, since there doesn't seem to be support for that right now). I've put down a few to-do tasks, lmk which ones you can take up and if there's anything missing from that list. |
Nice! The lookup tables are included inline in the yaml file now, no more txt files seperately. I can add those. I guess it makes sense to work in the order of the todos since they each depend on the previous to some degree. Would it be more helpful if I took an earlier one or a later one? |
Yah, can you take on the story conversion and test out the Core converter tool? I can work on the Forms migration in parallel since those changes will mostly be in the domain file. |
sounds good! |
Data validation CI check is failing because of a bug with the new ResponseSelector format (RasaHQ/rasa#6728) |
@b-quachtran I'm seeing errors running |
Intent Cross-Validation Results (3 folds)
Entity Cross-Validation Results (5 folds)
|
@melindaloubser1 I'm seeing a bunch of story contradictions being thrown during training, working through them at the moment but probably won't be resolved until Monday. |
Intent Cross-Validation Results (3 folds)
Entity Cross-Validation Results (5 folds)
|
Intent Cross-Validation Results (3 folds)
Entity Cross-Validation Results (5 folds)
|
@melindaloubser1 Context switching is working now and the rule conflicts are resolved. Can you review when you get a chance? |
Will do. Have we opened an issue for the nlu_fallback confusion during evaluation anywhere? |
Intent Cross-Validation Results (3 folds)
Entity Cross-Validation Results (5 folds)
|
Yeah it's in the discussion in the issue Akela made here |
Make pronouns general where they have no reason to be specific.
Just to make folks aware. I just tried running
It's suggested here that it might be due to a data conversion error so I figured I'd check in to let folks know. This error occurs after the training phase is done, just before it starts with the validation bit. EDIT It seems to also fail by just calling |
@koaning I think this might be an environment thing, I'm not seeing that on 2.0.2 |
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.
besides the false positive story conflicts (after ...
and after action_listen
) this looks good!
Intent Cross-Validation Results (3 folds)
Entity Cross-Validation Results (5 folds)
|
Intent Cross-Validation Results (3 folds)
Entity Cross-Validation Results (5 folds)
|
Intent Cross-Validation Results (3 folds)
Entity Cross-Validation Results (5 folds)
|
Converting Rasa-Demo to 2.0 Format
To do: