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

sentencepiece==0.1.92 causing segmentation fault #4857

Closed
boy2000-007man opened this issue Jun 9, 2020 · 12 comments
Closed

sentencepiece==0.1.92 causing segmentation fault #4857

boy2000-007man opened this issue Jun 9, 2020 · 12 comments
Labels
dependencies Pull requests that update a dependency file

Comments

@boy2000-007man
Copy link
Contributor

🐛 Bug

Information

transformers==2.9.1
torch==1.4.0

Starting from today,
I notice newly released sentencepiece==0.1.92 causing segmentation fault while calling torch functions.
Downgrade to sentencepiece==0.1.91 solve it.

@BramVanroy BramVanroy added the dependencies Pull requests that update a dependency file label Jun 9, 2020
@WencongXiao
Copy link

@boy2000-007man Hi, folk. Just curious about how do you find this bug? It costs me almost the whole day... Anyway, thank you so much!

@liesun1994
Copy link

OMG!!! Awesome Advice!!!!

@hahahawu
Copy link

I spent a whole night to address the dependencies problems and almost lost my mind. This answer saved my life. Appreciate!

@iamshwin
Copy link

Thanks for this! Also curious how you worked this out - I've spent a whole day trying to figure this out!

@libing125
Copy link

Thanks so much you saved my day.

@aychang95
Copy link

aychang95 commented Jun 26, 2020

I was dreading the thought of having to dive into this issue with faulthandler and meticulously cross referencing dependencies with a working version....but this post just saved my night. Thanks @boy2000-007man

This seems like a new pytorch v1.4.0 incompatibility issue with the latest huggingface releases. I'm assuming this may have been missed due to the focus on v1.5.0 support, but it seems like many people cannot make the jump to cuda 10.2/pytorch 1.5.0 currently, so this seems like a pretty big headache that should be addressed.

@LysandreJik
Copy link
Member

Closing this as solved by #5418

@ShaneTian
Copy link

You are excellent!

@zhjunqin
Copy link

same problem when use sentencepiece==0.1.94

@jayparks
Copy link

jayparks commented Nov 4, 2020

Having the same problem with sentencepiece==0.1.94

@thomwolf
Copy link
Member

thomwolf commented Nov 4, 2020

Cf #8199 we will remove the hard dependency on sentencepiece (replaced by the tokenizers library) in a coming release, probably end of next week.

@ShengdingHu
Copy link

Thank you a lot! You saved my day!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

No branches or pull requests