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.

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

UnsupportedModelError could be made clearer. #7069

Closed
koaning opened this issue Oct 20, 2020 · 1 comment
Closed

UnsupportedModelError could be made clearer. #7069

koaning opened this issue Oct 20, 2020 · 1 comment
Assignees
Labels
area:rasa-oss 🎡 Anything related to the open source Rasa framework type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR

Comments

@koaning
Copy link
Contributor

koaning commented Oct 20, 2020

I was using Rasa 1.10 to load in a Rasa 2.0 model. This was accidental on my part, but the error message seems to assume that you're loading in an "older" model. It is also throw when you're loading in a newer model.

Here's the error I got.

UnsupportedModelError: The model version is too old to be loaded by this Rasa NLU instance. Either retrain the model, or run with an older version. Model version: 1.10.3 Instance version: 2.0.0

Would anybody oppose if we change it to;

UnsupportedModelError: The model was trained using Rasa 1.10.3 and does not match your Rasa installation (2.0.0). This means that you probably want to retrain your model to ensure that the versions match up again. 

I'll gladly pick it up when I get a green light.

@koaning koaning added type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors. area:rasa-oss 🎡 Anything related to the open source Rasa framework labels Oct 20, 2020
@alwx alwx added type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR and removed type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors. labels Jan 27, 2021
@TyDunn
Copy link
Contributor

TyDunn commented Jan 27, 2021

@koaning @alwx and I discussed that this makes sense, so I think you can go ahead 👍

@koaning koaning self-assigned this Jan 27, 2021
@m-vdb m-vdb closed this as completed Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:rasa-oss 🎡 Anything related to the open source Rasa framework type:enhancement ✨ Additions of new features or changes to existing ones, should be doable in a single PR
Projects
None yet
Development

No branches or pull requests

4 participants