Skip to content

Conversation

@vblagoje
Copy link
Member

@vblagoje vblagoje commented Aug 25, 2022

Upgrades and pins transformers to 4.21.2

The most important change for us allowing "mps" devices in pipelines
After upgrading and pinning to 4.21.2, we can complete #3062

How did you test it?

We'll check whether any tests fail, for example, due to deprecated/discontinued features or breaking changes. We'll also comb through the changelog and check for any changes that might affect Haystack.

@vblagoje vblagoje requested a review from a team as a code owner August 25, 2022 09:35
@vblagoje vblagoje requested review from bogdankostic and removed request for a team August 25, 2022 09:35
@vblagoje vblagoje changed the title Upgrade and pin transformers to 4.21.2 chore: Upgrade and pin transformers to 4.21.2 Aug 29, 2022
@vblagoje vblagoje requested a review from julian-risch August 29, 2022 09:54
Copy link
Member

@julian-risch julian-risch left a comment

Choose a reason for hiding this comment

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

Looks good to me! 👍 I am very much looking forward to this as should be anybody using M1 or M2. 🙂 I had a brief look through transformers release notes but didn't see anything that could cause trouble: https://github.com/huggingface/transformers/releases
There were some bugs in 4.21.0 but they have been fixed in the two patch releases since then.

@vblagoje vblagoje merged commit 99efab7 into deepset-ai:main Aug 29, 2022
@vblagoje vblagoje deleted the t_upgrade_hf branch February 28, 2023 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants