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

Bert model with AlbertTokenizer problem #10584

Closed
ganbaaelmer opened this issue Dec 27, 2021 · 5 comments
Closed

Bert model with AlbertTokenizer problem #10584

ganbaaelmer opened this issue Dec 27, 2021 · 5 comments
Labels
area:rasa-oss 🎡 Anything related to the open source Rasa framework type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors.

Comments

@ganbaaelmer
Copy link

ganbaaelmer commented Dec 27, 2021

Rasa Open Source version

2.8.16

Rasa SDK version

2.8.3

Rasa X version

0.42.6

Python version

3.8

What operating system are you using?

Linux

What happened?

the error came from a HF model using a different tokenizer (e.g. BertModel with AlbertTokenizer), breaks due to the hard coded mapping in registry.py.

https://forum.rasa.com/t/how-to-import-huggingface-models-to-rasa/50238

Command / Request

No response

Relevant log output

No response

@ganbaaelmer ganbaaelmer added area:rasa-oss 🎡 Anything related to the open source Rasa framework type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors. labels Dec 27, 2021
@sara-tagger
Copy link
Collaborator

Thanks for raising this issue, @aeshky will get back to you about it soon✨

Please also check out the docs and the forum in case your issue was raised there too 🤗

@ganbaaelmer
Copy link
Author

@aeshky please help us

@mleimeister
Copy link
Contributor

This is not a bug but rather a known limitation of the current implementation. The documentation has been updated to clarify which models work with the current version of LanguageModelFeaturizer here: #10385
Exploratory work on how it can be improved and allow loading arbitrary models is already happening (#6307, #10624), therefore this ticket could be closed imo.

@aeshky
Copy link
Contributor

aeshky commented Jan 10, 2022

Thanks for looking into this @mleimeister. I'm closing this issue based on your comment.

@aeshky aeshky closed this as completed Jan 10, 2022
@mleimeister
Copy link
Contributor

@ganbaaelmer Please follow this ticket for further updates on the progress.

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:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors.
Projects
None yet
Development

No branches or pull requests

4 participants