Skip to content

Conversation

@wkpark
Copy link
Contributor

@wkpark wkpark commented Oct 25, 2023

Description

  • call state.jobnext() before postprocess*() to support job_count, job_no controls in postprocessing easily.

with this small fix, state.job_count and state.job_no could be easily controlled without hacky ways.

  • without this fix, one can control job_count and job_no in postprocess*() by the following hacky way:
    • increase job_no in the postprocess*() - HACK
    • increase job_count in the postprocess* (slight jumping back progress bar)
    • decrease job_no before return. (possible live progress bar jumping back too much will occur) - HACK

Show case (with or without this fix. above hacky workaround applied)

  • webui + a face detailer extension with progress support (postprocess extension, use postprocess_image() internally)
jum.back3.mp4

Checklist:

@wkpark wkpark requested a review from AUTOMATIC1111 as a code owner October 25, 2023 19:22
@AUTOMATIC1111 AUTOMATIC1111 merged commit 21d5618 into AUTOMATIC1111:dev Nov 3, 2023
@w-e-w w-e-w mentioned this pull request Dec 4, 2023
@w-e-w w-e-w mentioned this pull request Dec 16, 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.

2 participants