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

[Bug]: hires.fix denoising strength always saved as 0.4 value in image metadata if ADetailer is active #552

Closed
himmelweiss opened this issue Mar 24, 2024 · 7 comments · Fixed by #569
Labels
bug Something isn't working

Comments

@himmelweiss
Copy link

Describe the bug

ADetailer version: 24.3.2

If hires.fix is enabled, hires.fix denoising strength will be "always" saved as 0.4 value in the metadata of the result image (no matter what i set the value to originally).

As soon i disable ADetailer, hires.fix denoising value is saved with the correct value.

Screenshots

No response

Console logs, from start to end.

Python 3.10.6 (tags/v3.10.6:9c7b4bd, Aug  1 2022, 21:53:49) [MSC v.1932 64 bit (AMD64)]
Version: v1.8.0
Commit hash: bef51aed032c0aaa5cfd80445bc4cf0d85b408b5
#######################################################################################################
Initializing Civitai Link
If submitting an issue on github, please provide the below text for debugging purposes:

Python revision: 3.10.6 (tags/v3.10.6:9c7b4bd, Aug  1 2022, 21:53:49) [MSC v.1932 64 bit (AMD64)]
Civitai Link revision: 115cd9c35b0774c90cb9c397ad60ef6a7dac60de
SD-WebUI revision: bef51aed032c0aaa5cfd80445bc4cf0d85b408b5

Checking Civitai Link requirements...

#######################################################################################################
Launching Web UI with arguments: --theme=dark --xformers
[-] ADetailer initialized. version: 24.3.2, num models: 24

List of installed extensions

No response

@himmelweiss himmelweiss added the bug Something isn't working label Mar 24, 2024
@tetsuoo-online
Copy link

tetsuoo-online commented Mar 25, 2024

I came for the same issue, I can confirm it's happening for me as well, I always use external TXT files but it's in the image metadata as well, always stuck at 0.4 its very annoying I can't compare generated images anymore or reproduce them accurately

@Bing-su
Copy link
Owner

Bing-su commented Mar 25, 2024

I haven't cared about metadata for a long time, so I'm not sure, how long has this been an issue?

@himmelweiss
Copy link
Author

I haven't cared about metadata for a long time, so I'm not sure, how long has this been an issue?

Since quite some time so it seems 😄
Only noticed it just yesterday.

@miasik
Copy link

miasik commented Mar 30, 2024

I haven't cared about metadata for a long time, so I'm not sure, how long has this been an issue?

Maybe about a week ago

@Bing-su
Copy link
Owner

Bing-su commented Apr 6, 2024

The image that is processed through adetailer becomes a product of img2img as a result. Right now, this seems like a tough problem to solve, since denoising strength in hires and denoising strength in img2img have the same name.

@himmelweiss
Copy link
Author

The image that is processed through adetailer becomes a product of img2img as a result. Right now, this seems like a tough problem to solve, since denoising strength in hires and denoising strength in img2img have the same name.

And there is no way to catch the hires.fix denoising strength value and send it over to the img2img process?
Shouldn't matter if the name is the same? Or maybe i am just misunderstanding something there, quite possible as i have no idea about the overall code and how everything works etc., too lazy to dig through all this, i have my fair share of code annoyances already in my job since several decades :D

Nakadun added a commit to Nakadun/adetailer that referenced this issue Apr 9, 2024
@Nakadun
Copy link
Contributor

Nakadun commented Apr 9, 2024

Oh should have made the request to the dev branch. My bad :|

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants