Skip to content

[WIP] LEdits++ refactoring#7318

Draft
manuelbrack wants to merge 2 commits intohuggingface:mainfrom
ml-research:ledits_pp
Draft

[WIP] LEdits++ refactoring#7318
manuelbrack wants to merge 2 commits intohuggingface:mainfrom
ml-research:ledits_pp

Conversation

@manuelbrack
Copy link
Copy Markdown
Contributor

Make some code quality improvements to the LEdits++ pipelines

During the work on the initial Ledits++ PR, @yiyixuxu and @patrickvonplaten made some additional suggestions that will improve code quality. That refactoring will be done in this PR.

TODOS

  • Adjust check_inputs method of both pipelines to account for LEdits++ functionality
  • Refactor out SEGA/LEdits++ code of diffusion loop in __call__

@manuelbrack manuelbrack changed the title LEdits++ refactoring [WIP] LEdits++ refactoring Mar 14, 2024
@yiyixuxu
Copy link
Copy Markdown
Collaborator

@manuelbrack
thanks for the PR
yeah I think these two are the main to-dos we can start with :)

@github-actions
Copy link
Copy Markdown
Contributor

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread.

Please note that issues that do not follow the contributing guidelines are likely to be ignored.

@github-actions github-actions Bot added the stale Issues that haven't received updates label Apr 13, 2024
@linoytsaban linoytsaban removed stale Issues that haven't received updates labels Jun 8, 2024
@github-actions
Copy link
Copy Markdown
Contributor

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread.

Please note that issues that do not follow the contributing guidelines are likely to be ignored.

@github-actions github-actions Bot added the stale Issues that haven't received updates label Sep 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale Issues that haven't received updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants