-
Notifications
You must be signed in to change notification settings - Fork 4.8k
Regr Test: Fix dataset name bug #10935
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
Conversation
Status of the run: Failed Commit: 63ed36e, The full report is available as an artifact. |
Status of the run: Succeeded Commit: 63ed36e, The full report is available as an artifact. |
Status of the run: Succeeded Commit: 804045b, The full report is available as an artifact. |
Status of the run: Succeeded Commit: ffba9e1, The full report is available as an artifact. |
1594521
to
f4006b1
Compare
Status of the run: Failed Commit: 1fb904d, The full report is available as an artifact. Status of the run: Succeeded Commit: 1fb904d, The full report is available as an artifact. |
Hey @markus-hinsche! 👋 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 include:
- dataset: ["financial-demo"]
config: ["Sparse + BERT + 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. |
aa536c5
to
2ea7ce4
Compare
2ea7ce4
to
64e08a1
Compare
Problem description:
dataset
tag associated to custom metric sent to Datadog contained dataset path (e.g.RasaHQ/financial-demo
instead of the dataset name (e.g.financial-demo
). This was accidentally introduced here: Regr test: Refactor metrics and tags #10856before:
Proposed changes:
financial-demo
) instead of dataset path (e.g.RasaHQ/financial-demo
)Status (please check what you already did):
black
(please check Readme for instructions)