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

Updated training_example.ipynb #484

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dinarkino
Copy link

What does this PR do?

The original notebook doesn't work out of the box. After the following fixes, it works.
Fixes

  • updated diffusers[training] version to the 0.27.2
  • fixed "TODO" in one of the cells by adding a missing link to another notebook with Unconditional image generation
  • changed logging_dir to project_dir, because that was a reason for the error (this is also mentioned here: Fixed variable name #412)
  • added the create_repo function for the creation of a repository for the model save if we choose to upload the model to the Hugging Face Hub. Before that, it caused an error during repository pulling

Who can review?

@anton-l

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

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.

1 participant