fix: modify create_output_panel output to match object returned

[jarrodpan]

Ever since this commit AUTOMATIC1111/stable-diffusion-webui@501993e#diff-7825c63da20059aa9e36a44157ab1a6b36a02bd71c01f5f30ea5d028320ef263R240

the signature of create_output_panel has been a class object which can not be iterated, however ui_right.py expects an iterable.

This PR fixes the issue in a similar style to https://github.com/hako-mikan/sd-webui-supermerger/pull/323/files

fixes #949 #948 #934

[jarrodpan]

A similar fix is present here https://github.com/deforum-art/sd-forge-deforum/blob/main/scripts/deforum_helpers/ui_right.py#L99-L105

[walteryoung]

Confirmed that this fixed the issue locally for me. Thanks

[viking1304]

This only partially works on my computer. Clicking on the Generate button does nothing. 🤷🏻‍♂️

But, as soon as I added a new _interrupting element and the handler, everything started to work.

interrupting = gr.Button('Interrupting...', elem_id=f"{id_part}_interrupting", elem_classes="generate-box-interrupting", tooltip="Interrupting generation...")

interrupting.click(
  fn=lambda: state.interrupt(),
  inputs=[],
  outputs=[],
)

I am on A1111 1.8.0

commit bef51aed032c0aaa5cfd80445bc4cf0d85b408b5 (HEAD -> master, tag: v1.8.0, origin/master, origin/HEAD)
Merge: cf2772fa 13984857
Author: AUTOMATIC1111 <16777216c@gmail.com>
Date:   Sat Mar 2 07:03:13 2024 +0300

    Merge branch 'release_candidate'

[walteryoung]

This only partially works on my computer. Clicking on the Generate button does nothing. 🤷🏻‍♂️

But, as soon as I added a new _interrupting element and the handler, everything started to work.

interrupting = gr.Button('Interrupting...', elem_id=f"{id_part}_interrupting", elem_classes="generate-box-interrupting", tooltip="Interrupting generation...")

interrupting.click(
  fn=lambda: state.interrupt(),
  inputs=[],
  outputs=[],
)

I am on A1111 1.8.0

commit bef51aed032c0aaa5cfd80445bc4cf0d85b408b5 (HEAD -> master, tag: v1.8.0, origin/master, origin/HEAD)
Merge: cf2772fa 13984857
Author: AUTOMATIC1111 <16777216c@gmail.com>
Date:   Sat Mar 2 07:03:13 2024 +0300

    Merge branch 'release_candidate'

@viking1304 What lines do you add the interrupting code on?

[viking1304]

@walteryoung I am not sure I understood your question correctly.

If it was about what my code was based on, then this A1111 commit
If your question was where to put it, here it is.

[viking1304]

@jarrodpan please merge my pull request for buttons

[jarrodpan]

@jarrodpan please merge my pull request for buttons

Thanks for this, I can confirm i had the same issue and your changes work, i'll merge them into this PR.

[rewbs]

Thanks! I had fixed this for Forge in https://github.com/deforum-art/sd-forge-deforum but didn't realise it also applied to recent a1111 versions.

[kylewetton]

Are you sure object_panel has gallery as an option?

*** Error executing callback ui_tabs_callback for C:\Users\krwet\stable-diffusion-webui\extensions\sd-webui-deforum\scripts\deforum.py
    Traceback (most recent call last):
      File "C:\Users\krwet\stable-diffusion-webui\modules\script_callbacks.py", line 166, in ui_tabs_callback
        res += c.callback() or []
      File "C:\Users\krwet\stable-diffusion-webui\extensions\sd-webui-deforum\scripts\deforum_helpers\ui_right.py", line 101, in on_ui_tabs
        deforum_gallery = output_panel.gallery
    AttributeError: 'tuple' object has no attribute 'gallery'

---

[viking1304]

@kylewetton you need to update a1111 to 1.8.0 and then to update deforum, and everyting should work

[andyxr]

So, we ready to release this onto the world? People need... :-)

[andyxr]

Ah it's been merged. I'm obviously having a bad hair day!