Skip to content

fix 10896 pnginfo parameters#10905

Merged
AUTOMATIC1111 merged 2 commits into
devfrom
fix-10896-pnginfo-parameters
Jun 1, 2023
Merged

fix 10896 pnginfo parameters#10905
AUTOMATIC1111 merged 2 commits into
devfrom
fix-10896-pnginfo-parameters

Conversation

@w-e-w

@w-e-w w-e-w commented May 31, 2023

Copy link
Copy Markdown
Collaborator

Description

fix #10896
remove the two lines that causes the parameters to be overwritten by upscaling

to be honest I don't know what the point of existing_pnginfo['parameters'] = geninfo it seems to only cause trouble
from what I can tell when generate images (txt2img)
existing_pnginfo['parameters'] always equals geninfo so there's no point of reassigning it
and when you upscaling in extra tab, that lines essentially overrides the generation parameters

as far I can tell this change the final pnginfo back to what it was like in 1.2.1

you may want to do some testing yourself to make sure that I didn't overlook something

@w-e-w w-e-w requested a review from AUTOMATIC1111 as a code owner May 31, 2023 21:29
@w-e-w w-e-w linked an issue May 31, 2023 that may be closed by this pull request
1 task
@AUTOMATIC1111 AUTOMATIC1111 merged commit e980a4b into dev Jun 1, 2023
@AUTOMATIC1111 AUTOMATIC1111 deleted the fix-10896-pnginfo-parameters branch June 1, 2023 03:54
@w-e-w

w-e-w commented Jun 2, 2023

Copy link
Copy Markdown
Collaborator Author

#10940

AUTOMATIC1111 added a commit that referenced this pull request Jun 2, 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.

[Bug]: Image Parameters arent stored anymore when sending to extras and upscaling.

2 participants