-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Integrate Graph Architecture with rest of codebase #9803
Conversation
…ipe' into 3.0-architecture-revamp/9604/train-and-run
…chitecture-revamp/9604/train
Some model regression tests were failing over night due to poetry issues. As we have also seen no changes in previous runs and the entity testing works now, I just give it one final run on the Sara dataset before merging @joejuzl @ka-bu |
Commit: acd4601, The full report is available as an artifact. Dataset:
|
Commit: df44e56, The full report is available as an artifact. Dataset:
|
Run when compared to # 134 (last fully successful model reg. test) https://github.com/RasaHQ/rasa/actions/runs/1111032086 Dataset:
Dataset:
Dataset:
Dataset:
Dataset:
Dataset:
|
Run when compared to TF 2.6 (note 11% drop for Rules+Memo for Sara is not explained yet, ignore this) Dataset:
Dataset:
Dataset:
Dataset:
Dataset:
Dataset:
|
Hey @wochinge! 👋 To run model regression tests, comment with the Tips 💡: The model regression test will be run on Tips 💡: Every time when you want to change a configuration you should edit the comment with the previous configuration. You can copy this in your comment and customize:
|
/modeltest dataset_branch: "architecture-revamp/modified-spacy-configs"
include:
- dataset: ["Carbon Bot"]
config:
- "BERT + DIET(bow) + ResponseSelector(bow)"
- "BERT + DIET(seq) + ResponseSelector(t2t)"
- "Spacy + DIET(bow) + ResponseSelector(bow)"
- "Spacy + DIET(seq) + ResponseSelector(t2t)"
- "Sparse + BERT + DIET(bow) + ResponseSelector(bow)"
- "Sparse + BERT + DIET(seq) + ResponseSelector(t2t)"
- "Sparse + DIET(bow) + ResponseSelector(bow)"
- "Sparse + DIET(seq) + ResponseSelector(t2t)"
- "Sparse + Spacy + DIET(bow) + ResponseSelector(bow)"
- "Sparse + Spacy + DIET(seq) + ResponseSelector(t2t)"
|
The model regression tests have started. It might take a while, please be patient. Used configuration can be found in the comment. |
Commit: 58cc760, The full report is available as an artifact. Dataset:
|
Proposed changes:
Recipe
to convert current config toGraphSchema
#9277rasa train
andrasa run
with new model architecture #9604Open Issues
Status (please check what you already did):
black
(please check Readme for instructions)