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

add "Use ADetailer only for hires-fix" setting #760

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

david419kr
Copy link

안녕하세요, 개인적으로 있으면 좋겠다 싶은 기능이라서 settings에 "Hires Fix를 이용할 때만 ADetailer가 적용됨"을 추가해 보았습니다.
말 그대로, 일반 이미지 생성시에는 ADetailer가 enabled 상태여도 ADetailer가 적용되지 않으며,
Hires Fix가 켜져있거나, 혹은 일반 이미지 생성 후 이미지 우하단 별표 Upscale 버튼을 눌렀을 때만 ADetailer가 적용되도록 하는 옵션입니다.

image

default 값은 False이며, 해당 세팅이 켜져있을 시 Hires Fix가 아닌 일반 이미지 생성시 아래와 같은 안내문이 콘솔에 뜨도록 하였습니다.

image

혹시 괜찮으시다면 merge 고려해주시면 감사합니다!

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jan 2, 2025
Copy link

sonarqubecloud bot commented Jan 2, 2025

@MisterChief95
Copy link

That's interesting, I added similar functionality to the extension on my machine, however, I added it at the top level within the extension dropdown, like so:
image

Now I'm curious which approach may be preferred.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants