Skip to content

[docs] Callbacks - #5735

Merged
stevhliu merged 3 commits into
huggingface:mainfrom
stevhliu:callback-docs
Nov 13, 2023
Merged

[docs] Callbacks#5735
stevhliu merged 3 commits into
huggingface:mainfrom
stevhliu:callback-docs

Conversation

@stevhliu

@stevhliustevhliu commented Nov 9, 2023

Copy link
Copy Markdown
Member

Follow-up PR to #5427:

  • updates the title and moves it closer in the toctree to the other pipeline-adjacent usage docs
  • starts with a bit more context about what you can potentially do with callbacks to highlight its flexibility
  • fixes minor typo in the API docstrings

@HuggingFaceDocBuilderDev

HuggingFaceDocBuilderDev commented Nov 9, 2023•
edited
Loading

Copy link
Copy Markdown

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

yiyixuxu
yiyixuxu approved these changes Nov 11, 2023
View reviewed changes

@yiyixuxuyiyixuxu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thank you!

Comment threaddocs/source/en/using-diffusers/callback.md Outdated
patrickvonplaten
patrickvonplaten approved these changes Nov 13, 2023
View reviewed changes

@patrickvonplatenpatrickvonplaten left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Very nice!

@stevhliu
stevhliu merged commit f782ca1 into huggingface:mainNov 13, 2023
@stevhliu
stevhliu deleted the callback-docs branch November 13, 2023 20:11
yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this pull request Dec 25, 2023
@stevhliu
[docs] Callbacks (huggingface#5735)
f52abe6
* updates
* feedback
AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this pull request Apr 26, 2024
@stevhliuJimmy
[docs] Callbacks (huggingface#5735)
2f06746
* updates
* feedback
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yiyixuxuyiyixuxuyiyixuxu approved these changes
+1 more reviewer
@patrickvonplatenpatrickvonplatenpatrickvonplaten approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@stevhliu@HuggingFaceDocBuilderDev@yiyixuxu@patrickvonplaten