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

TF CLM example fix typo #13002

Merged
merged 1 commit into from Aug 31, 2021
Merged

TF CLM example fix typo #13002

merged 1 commit into from Aug 31, 2021

Conversation

Rocketknight1
Copy link
Member

Fixes a one-line typo in the TF CLM example - it was still using MODEL_FOR_MASKED_LM_MAPPING

@Rocketknight1
Copy link
Member Author

Also oop, this looks like it was based on a slightly older branch - it won't cause any problems, but the "Files changed" tab lists some changes in other files that are already merged - the only one actually affected is run_clm.py

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.

Damned merge commit... Thanks for fixing!

@Rocketknight1 Rocketknight1 merged commit 702f4a4 into master Aug 31, 2021
@Rocketknight1 Rocketknight1 deleted the tf_clm_example_fix branch August 31, 2021 12:21
@cakiki
Copy link
Contributor

cakiki commented Oct 21, 2021

There might be more such remnants from the MLM script, see #14014

cc @Rocketknight1

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.

None yet

3 participants