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

PyTorch >= 1.7.0 and TensorFlow >= 2.4.0 #19016

Merged
merged 1 commit into from
Sep 14, 2022
Merged

PyTorch >= 1.7.0 and TensorFlow >= 2.4.0 #19016

merged 1 commit into from
Sep 14, 2022

Conversation

sgugger
Copy link
Collaborator

@sgugger sgugger commented Sep 13, 2022

What does this PR do?

As discussed in #18817 we have decided that Transformers will support versions of PyTorch and TensorFlow for two years after their releases. As a result, Transformers can now assume a minimum version of 1.7.0 for PyTorch and 2.4.0 for TensorFlow. This PR enforces this in the setup and then simplifies a lot of code that was written to support older PyTorch versions.

Copy link
Member

@LysandreJik LysandreJik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, this is a very welcome end-of-summer cleaning 😄

LGTM.

Ok for you @ydshieh?

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Sep 13, 2022

The documentation is not available anymore as the PR was closed or merged.

Copy link
Collaborator

@ydshieh ydshieh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the housekeeping 🤗

@sgugger sgugger merged commit a2a3afb into main Sep 14, 2022
@sgugger sgugger deleted the pytorch_1_7 branch September 14, 2022 11:19
@corey-nm
Copy link

corey-nm commented Oct 7, 2022

@sgugger curious what the reasoning was for blacklisting torch 1.12.0? I looked in #18817, but there's no info there for 1.12

@sgugger
Copy link
Collaborator Author

sgugger commented Oct 7, 2022

That particular release broke weightnorm, and thus all audio models.

setu4993 added a commit to setu4993/convert-labse-tf-pt that referenced this pull request Dec 29, 2022
Since Sept, HF Transformers doesn't support 1.6 and below. huggingface/transformers#19016
setu4993 added a commit to setu4993/convert-labse-tf-pt that referenced this pull request Dec 29, 2022
Since Sept, HF Transformers doesn't support 1.6 and below. huggingface/transformers#19016
setu4993 added a commit to setu4993/convert-labse-tf-pt that referenced this pull request Dec 29, 2022
* chore: Update `torch` version

Since Sept, HF Transformers doesn't support 1.6 and below. huggingface/transformers#19016

* chore: Update to `pytest` v7

* chore: Move dev dependencies to group

* chore: Support `transformers` v4.22.0 and above only

* chore: Use py37 and above

* chore: Add `ipython` for dev

* style: Simplify isort config

* chore: Bump `loguru` to v0.6.0

* chore: Update GitHub workflows

* chore: Set `torch`, `tf`, `flax` as transient deps with HF transformers' extras

* chore: Add `tensorflow-text`

* chore: Update lock file

* chore: Add `sentencepiece` extras

* chore: Update lock file

* style: Apply updated `isort` config

* chore: Bump major version

Breaking change since we're dropping py37 support here.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants