Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

move upscale postprocessing under input accordion#15223

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

@light-and-ray
Copy link
Contributor

@light-and-raylight-and-ray commentedMar 11, 2024
edited
Loading

Description

Move upscaler script in extras under input accordion. It's opened and enabled by default

Why I've made this:

  • Upscaler 2 dropdown was at the same level withGFPGAN accordion
  • Scale by / Scale to tabs section looked detached form the other options. I often loose it from my view
  • to unify visual with other scripts

I've swapped scale row and upscalers rows for 2 reasons:

  • Move "heavier" element to the bottom
  • Repeat pattern fromHires. fix InputAccordion where Upscaler dropdown is upper then scale sliders

It's opened and enabled by default because:

  • it's main feature of extras tab
  • it preserves the previous behavior
  • if user hasn't overridden default upscaler, they can not close and disable it because of None default upscalers. (The minimal number of clicks)

I've chosen InputAccordion to allow user change default upscaler, and simple disable upscaling script if they doesn't want to use it

Screenshots/videos:

Before:
Screenshot 2024-03-12 at 02-06-50 Stable Diffusion

After:
Screenshot 2024-03-12 at 02-12-18 Stable Diffusion

Checklist:

@w-e-w
Copy link
Collaborator

second this
I've been meaning to do this myself but just never got around doing it

@AUTOMATIC1111AUTOMATIC1111 merged commit2fc47b4 intoAUTOMATIC1111:devMar 16, 2024
@light-and-raylight-and-ray deleted the move_upscale_postprocessing_under_input_accordion branchMarch 16, 2024 08:35
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@AUTOMATIC1111AUTOMATIC1111AUTOMATIC1111 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@light-and-ray@w-e-w@AUTOMATIC1111

[8]ページ先頭

©2009-2025 Movatter.jp