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

Deprecate init image correctly #1649

Merged
merged 1 commit into from
Dec 12, 2022

Conversation

patrickvonplaten
Copy link
Contributor

@patrickvonplaten patrickvonplaten commented Dec 10, 2022

Currently one has to pass image=... because it is a required argument. This doesn't correctly deprecate the use of init_image though.

@patrickvonplaten patrickvonplaten changed the title Deprecate init image correctl Deprecate init image correctly Dec 10, 2022
@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Dec 10, 2022

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

@patrickvonplaten patrickvonplaten requested review from patil-suraj, pcuenca and anton-l and removed request for patil-suraj and pcuenca December 10, 2022 16:11
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.

Thanks for fixing!

image: Union[np.ndarray, PIL.Image.Image],
mask_image: Union[np.ndarray, PIL.Image.Image],
image: Union[np.ndarray, PIL.Image.Image] = None,
mask_image: Union[np.ndarray, PIL.Image.Image] = None,
Copy link
Member

Choose a reason for hiding this comment

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

mask_image wan't deprecated, was it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No but I don't want to change the order here - if image becomes a keyword argument mask image has to become one as well.

SyntaxError: non-default argument follows default argument

@patrickvonplaten patrickvonplaten merged commit 8bf5e59 into main Dec 12, 2022
@patrickvonplaten patrickvonplaten deleted the make_sure_init_image_can_be_used branch December 12, 2022 14:00
tcapelle pushed a commit to tcapelle/diffusers that referenced this pull request Dec 12, 2022
sliard pushed a commit to sliard/diffusers that referenced this pull request Dec 21, 2022
yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this pull request Dec 25, 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.

None yet

3 participants