Skip to content

Conversation

@sgugger
Copy link
Collaborator

@sgugger sgugger commented Dec 5, 2022

What does this PR do?

The test for file not found in the TensorFlow auto model tests is failing on main as the message does not match exactly (see here). This PR fixes that.

@sgugger sgugger requested a review from ydshieh December 5, 2022 23:11
@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Dec 5, 2022

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

@sgugger sgugger merged commit 5764efe into main Dec 5, 2022
@sgugger
Copy link
Collaborator Author

sgugger commented Dec 5, 2022

Merging to have tests passing on main, but I will address any comment in followup PRs :-)

@sgugger sgugger deleted the fix_tf_auto_test branch December 5, 2022 23:33
with self.assertRaisesRegex(
EnvironmentError,
"hf-internal-testing/config-no-model does not appear to have a file named tf_model.h5",
"hf-internal-testing/config-no-model does not appear to have a file named pytorch_model.bin",
Copy link
Collaborator

@ydshieh ydshieh Dec 6, 2022

Choose a reason for hiding this comment

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

Good to see this fixed the CI, however this is somehow confusing: it is TFAutoModel, but seeing the expected error being just pytorch_model.bin here blows my mind.

(I know the full error is hf-internal-testing/config-no-model does not appear to have a file named pytorch_model.bin, tf_model.h5 or model.ckpt)

mpierrau pushed a commit to mpierrau/transformers that referenced this pull request Dec 15, 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