Skip to content

Conversation

@pcuenca
Copy link
Member

@pcuenca pcuenca commented Oct 17, 2022

This resulted in repos with no model files such as: https://huggingface.co/pcuenq/ddpm-ema-faces-64-50/tree/main

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Oct 17, 2022

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

Copy link
Contributor

@patrickvonplaten patrickvonplaten left a comment

Choose a reason for hiding this comment

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

Yes this makes sense to me! We should always save the pipeline in the local output dir no matter if we push to the Hub or not.

Copy link
Member

@anton-l anton-l left a comment

Choose a reason for hiding this comment

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

My oversight, thanks for fixing!

@pcuenca pcuenca merged commit 4dce374 into main Oct 17, 2022
@pcuenca pcuenca deleted the fix-no-model-pushed-to-hub branch October 17, 2022 13:29
kumquatexpress pushed a commit to harvestlabs/diffusers that referenced this pull request Oct 19, 2022
…e not saved before pushing to hub (huggingface#868)

Fix: models were not saved before pushing to hub.
prathikr pushed a commit to prathikr/diffusers that referenced this pull request Oct 26, 2022
…e not saved before pushing to hub (huggingface#868)

Fix: models were not saved before pushing to hub.
PhaneeshB pushed a commit to nod-ai/diffusers that referenced this pull request Mar 1, 2023
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.

5 participants