Skip to content

Conversation

@williamberman
Copy link
Contributor

No description provided.

Comment on lines 138 to 140
def test_inference_batch_single_identical(self):
self._test_inference_batch_single_identical(relax_max_difference=False)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is calling the helper with the default function argument, so don't need to override mixin function

@williamberman williamberman force-pushed the attend_and_excite_test_batch_timeouts branch from 6c25e34 to dde7095 Compare February 26, 2023 03:16
@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Feb 26, 2023

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

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.

Do we have any idea on the upper bound that causes things to timeout?

Copy link
Collaborator

@yiyixuxu yiyixuxu left a comment

Choose a reason for hiding this comment

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

In attend-and-exicte each latent is optimized individually so batching was done in a loop hense the timeout - I would love to know if there is a way to do it more efficiently too!

@williamberman williamberman merged commit 824cb53 into huggingface:main Feb 27, 2023
mengfei25 pushed a commit to mengfei25/diffusers that referenced this pull request Mar 27, 2023
AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this pull request Apr 26, 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