Skip to content

Fixed additional input rows. Added option for Schedule Type. #152

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

thebollo
Copy link

@thebollo thebollo commented Dec 5, 2024

Fixed additional input rows. Added option for Schedule Type.

@@ -347,11 +348,11 @@ def show(self, is_img2img):
return True

def ui(self, is_img2img):
try_init() # Moved to the beginning to ensure modes are registered before UI is built
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(A) it was already at the beginning, the move doesn't really make sense
(B) shouldn't leave change comments in code, make the change in code and leave notes on git

@@ -427,7 +428,7 @@ def on_axis_change(mode_name, out_file):
group_obj = sets[group + 1][0]
next_rows = sets[group + 1][1]
def make_vis(prior, r1, r2, r3, r4):
return gr.Group.update(visible=(prior+r1+r2+r3+r4).strip() != "")
return gr.update(visible=(prior+r1+r2+r3+r4).strip() != "")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why exactly is this change needed?

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