Skip to content

Conversation

@laksjdjf
Copy link
Contributor

What does this PR do?

Recreate from #7173

Who can review?

@sayakpaul , @yiyixuxu

@sayakpaul sayakpaul requested a review from yiyixuxu March 11, 2024 01:46
@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@sayakpaul
Copy link
Member

Let's fix the tests :)

@laksjdjf
Copy link
Contributor Author

Let's fix the tests :)

@sayakpaul
I removed meaningless spaces for test.
Are there any other operations required?

Copy link
Member

@sayakpaul sayakpaul left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@laksjdjf
Copy link
Contributor Author

@sayakpaul
Is this error relevant to this pr?

=========================== short test summary info ============================
FAILED tests/pipelines/unidiffuser/test_unidiffuser.py::UniDiffuserPipelineFastTests::test_attention_slicing_forward_pass - requests.exceptions.ReadTimeout: (ReadTimeoutError("HTTPSConnectionPool(host='huggingface.co', port=443): Read timed out. (read timeout=10)"), '(Request ID: 869e35f5-5627-4acf-9853-7187ce7d0656)')
==== 1 failed, 2193 passed, 989 skipped, 4415 warnings in 937.44s (0:15:37) ====

@sayakpaul
Copy link
Member

It's not!

@yiyixuxu yiyixuxu merged commit a6c9015 into huggingface:main Mar 19, 2024
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