Skip to content

Conversation

amyeroberts
Copy link
Collaborator

What does this PR do?

Fixes a small bug to make sure a TFAutoModel class keeps the TF naming pattern when being updated with auto_class_update.

Before submitting

Who can review?

Anyone in the community is free to review the PR once the tests have passed.

@amyeroberts amyeroberts requested a review from ydshieh April 28, 2022 15:06
@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Apr 28, 2022

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

@ydshieh
Copy link
Collaborator

ydshieh commented Apr 28, 2022

Thank you for the fix, @amyeroberts 🚀

@ydshieh ydshieh requested a review from sgugger April 28, 2022 15:30
Copy link
Collaborator

@sgugger sgugger left a comment

Choose a reason for hiding this comment

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

Thanks for fixing!

@ydshieh
Copy link
Collaborator

ydshieh commented Apr 28, 2022

@amyeroberts You can now merge the PR (Squash and merge button)

@amyeroberts amyeroberts merged commit 01562da into huggingface:main Apr 28, 2022
@amyeroberts amyeroberts deleted the update-tf-auto-model-name-to-reflect-pattern branch April 28, 2022 17:11
elusenji pushed a commit to elusenji/transformers that referenced this pull request Jun 12, 2022
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.

4 participants