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

Fix import error is_safetensors_available #407

Merged
merged 1 commit into from
Aug 18, 2023

Conversation

eaidova
Copy link
Collaborator

@eaidova eaidova commented Aug 18, 2023

What does this PR do?

fixes failure in tests with import error due to incompatible changes in diffusers 0.20.0

@eaidova
Copy link
Collaborator Author

eaidova commented Aug 18, 2023

@fxmarty @AlexKoff88 please take a look
it affects testing in #404 and #397

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Aug 18, 2023

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

@AlexKoff88
Copy link
Collaborator

Nice. Thanks, @eaidova! I've just asked if we need to update dependencies but it seems you made it more generic.

Copy link
Contributor

@fxmarty fxmarty left a comment

Choose a reason for hiding this comment

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

LGTM thank you!

@fxmarty fxmarty merged commit c70ad82 into huggingface:main Aug 18, 2023
9 of 10 checks passed
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