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

fix invalid params in stream_generator.py #150

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

JohnnyStreet
Copy link

_prepare_attention_mask_for_generation expects a config and kwargs but is being passed tensors here (which are accessible in the config anyway)

@eginhard
Copy link
Member

Thank you for following the development in transformers. That function was changed in huggingface/transformers@8a734ea which is not yet in a released version afaict. I think for Coqui it would be best to set an upper limit on transformers again (I'll do that in #149) and then after it has been released for some time we can adapt our code.

@eginhard eginhard added the blocked Blocked by something else label Nov 10, 2024
@eginhard eginhard marked this pull request as draft November 11, 2024 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Blocked by something else
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants