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

Upgrade transformers to fix CI #17830

Merged
merged 1 commit into from
Oct 9, 2023
Merged

Upgrade transformers to fix CI #17830

merged 1 commit into from
Oct 9, 2023

Conversation

snnn
Copy link
Member

@snnn snnn commented Oct 7, 2023

Description

Python package pipeline fails due to "tokenizers" compilation. Since "tokenizers" is a dep of "transformers", we update its version and hope a new solution had been there.

error: casting `&T` to `&mut T` is undefined behavior, even if the reference is unused, consider instead using an `UnsafeCell`
--> tokenizers-lib/src/models/bpe/trainer.rs:517:47

Motivation and Context

Cherry-pick from #17823

Python package pipeline fails due to "tokenizers" compilation. Since
"tokenizers" is a dep of "transformers", we update its version and hope
a new solution had been there.

```
error: casting `&T` to `&mut T` is undefined behavior, even if the reference is unused, consider instead using an `UnsafeCell`
--> tokenizers-lib/src/models/bpe/trainer.rs:517:47
```
@snnn snnn added the training issues related to ONNX Runtime training; typically submitted using template label Oct 7, 2023
@snnn
Copy link
Member Author

snnn commented Oct 8, 2023

/azp run Linux CPU CI Pipeline

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@snnn snnn merged commit 2a1fd25 into rel-1.16.1 Oct 9, 2023
189 of 191 checks passed
@snnn snnn deleted the snnn/rel-1.16.1 branch October 9, 2023 20:55
@snnn snnn mentioned this pull request Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
training issues related to ONNX Runtime training; typically submitted using template
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants