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

Separate prompt editing steps in hires fix #6817

Closed
1 task done
SirVeggie opened this issue Jan 16, 2023 · 3 comments
Closed
1 task done

Separate prompt editing steps in hires fix #6817

SirVeggie opened this issue Jan 16, 2023 · 3 comments
Labels
enhancement New feature or request

Comments

@SirVeggie
Copy link

SirVeggie commented Jan 16, 2023

Is there an existing issue for this?

  • I have searched the existing issues and checked the recent builds/commits

What happened?

Scenario: Generating an image at 50 steps, 10 highres fix steps.

If the prompt includes [cat:frog :0.2], the word cat changes to frog at 10 step mark. However during highres fix stage instead of changing at 2 step mark, it changes at 10 steps as well.

Steps to reproduce the problem

  1. prompt: [cat:frog :0.2]
  2. steps 50
  3. fix seed in place
  4. generate image
  5. enable highres fix at 10 steps, 0.6 denoise
  6. generate another image
  7. highres fix image is clearly more cat than the previous image

What should have happened?

When using percentage with prompt editing, it should be recalculated at the start of highres fix for the edit to occur in the correct position.

Commit where the problem happens

ff6a5bc

What platforms do you use to access UI ?

Windows

What browsers do you use to access the UI ?

Brave

Command Line Arguments

--xformers

Additional information, context and logs

image

@SirVeggie SirVeggie added the bug-report Report of a bug, yet to be confirmed label Jan 16, 2023
@SirVeggie SirVeggie changed the title [Bug]: Prompt editing steps are wrong with separate highres fix steps [Bug]: Prompt editing steps are wrong with separate hires fix steps Feb 6, 2023
@guzuligo
Copy link

I was about to report same problem. I guess Hires timer resets when Hires starts working.

Maybe there should be a checkbox to choose to reset or not.

@catboxanon
Copy link
Collaborator

catboxanon commented Aug 8, 2023

Would be resolved by #12350

This is also technically not a bug so I've reworded this and swapped the issue label.

@catboxanon catboxanon changed the title [Bug]: Prompt editing steps are wrong with separate hires fix steps Separate prompt editing steps in hires fix Aug 8, 2023
@catboxanon catboxanon added enhancement New feature or request and removed bug-report Report of a bug, yet to be confirmed labels Aug 8, 2023
@rubberbaron
Copy link

#12350 has been superseded by #12457. That change will make this kind of irrelevant.

For what it's worth, though, the code as of now (without #12457) does recompute the hires schedule to account for hires steps, so the change for [:0.2] does occur 20% through, e.g. 2 steps with 10 hires steps. Therefore the original bug reported here seems to have been fixed at some point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants