-
Notifications
You must be signed in to change notification settings - Fork 27.5k
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
jinja2 is a necessary dependency, but it is not currently specified #34397
Comments
transformers/src/transformers/utils/chat_template_utils.py Lines 362 to 366 in 30c76d5
|
And "Extension" is from jinja2 transformers/src/transformers/utils/chat_template_utils.py Lines 28 to 33 in 30c76d5
|
This is a design decision to keep the number of installed packages small. It's listed here: https://github.com/huggingface/transformers/blob/main/setup.py#L127. As you see, not even torch is installed by default: https://github.com/huggingface/transformers/blob/main/setup.py#L184, https://github.com/huggingface/transformers/blob/main/setup.py#L423 |
This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Please note that issues that do not follow the contributing guidelines are likely to be ignored. |
Thanks for flagging this. I was getting the 'Extension' undefined error and was quire confused until I found this. |
Likewise, I ran into the same problem - really would be wonderful if there was at least a specific error message that directed the user to add jinja2 as a dep! |
System Info
transformers
version: 4.45.2Who can help?
No response
Information
Tasks
examples
folder (such as GLUE/SQuAD, ...)Reproduction
I have addressed the issue: if Jinja2 is not installed, an ‘Extension’ undefined error will occur.
Expected behavior
NA
The text was updated successfully, but these errors were encountered: