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

[Feature]: support X/Y/Z plot #236

Closed
Tracked by #224 ...
xyzDist opened this issue Oct 21, 2023 · 8 comments · Fixed by #431
Closed
Tracked by #224 ...

[Feature]: support X/Y/Z plot #236

xyzDist opened this issue Oct 21, 2023 · 8 comments · Fixed by #431

Comments

@xyzDist
Copy link

xyzDist commented Oct 21, 2023

Expected behavior

Hi Dev,
is that possible to support X/Y/Z plot to test the parameters in both t2i and i2i?
thanks in advance!

@continue-revolution
Copy link
Owner

How do you actually want xyz plot given that output is a video?

@xyzDist
Copy link
Author

xyzDist commented Oct 21, 2023

Wedge setting value per video, it does works with regular parameters, like denoise. it will wedge out different videos properly, however the extension's parameters are not avalible in the dropdown list (like controlnet or adetailer). so I was wondering if it is possible. Thanks!
image

@xyzDist
Copy link
Author

xyzDist commented Oct 21, 2023

the purpose is to test out different setting value in a much easiest way. here is testing the denoise value for example.
image

00085-12345
00087-12345
00088-12345

@continue-revolution
Copy link
Owner

I understand. What kind of arguments in AnimateDiff do you want to test there?

@continue-revolution continue-revolution mentioned this issue Oct 21, 2023
15 tasks
@xyzDist
Copy link
Author

xyzDist commented Oct 21, 2023

Awesome!, Thanks a lot @continue-revolution.
Here are the arguments list I think is useful

  • latent power
  • latent scale
  • optional latent power of last frame
  • optional latent scale for last frame
  • interp X
  • context batch size
  • stride
  • overlap
  • motion module

@continue-revolution
Copy link
Owner

Then please wait patiently since there is a long update list at #224 and I'm feeling overwhelmed

@xyzDist
Copy link
Author

xyzDist commented Oct 21, 2023

take your time, thanks for the efforts.

@continue-revolution continue-revolution mentioned this issue Nov 8, 2023
22 tasks
@continue-revolution continue-revolution mentioned this issue Dec 13, 2023
15 tasks
@solitaryTian
Copy link

the purpose is to test out different setting value in a much easiest way. here is testing the denoise value for example. image

00085-12345 00085-12345 00087-12345 00087-12345 00088-12345 00088-12345

Hello, I want to test the video quality with different samplers using XYZ plot. But the results can not shows in a single picture or gif. So I want to ask you if your results with different denoise value show in a single picture or gif?
The following is my error information.

Traceback (most recent call last):
  File "/dfs/comicai/guanyu.zhao/stable-diffusion-webui/modules/call_queue.py", line 57, in f
    res = list(func(*args, **kwargs))
  File "/dfs/comicai/guanyu.zhao/stable-diffusion-webui/modules/call_queue.py", line 36, in f
    res = func(*args, **kwargs)
  File "/dfs/comicai/guanyu.zhao/stable-diffusion-webui/modules/txt2img.py", line 52, in txt2img
    processed = modules.scripts.scripts_txt2img.run(p, *args)
  File "/dfs/comicai/guanyu.zhao/stable-diffusion-webui/modules/scripts.py", line 601, in run
    processed = script.run(p, *script_args)
  File "/dfs/comicai/guanyu.zhao/stable-diffusion-webui/scripts/xyz_grid.py", line 739, in run
    processed = draw_xyz_grid(
  File "/dfs/comicai/guanyu.zhao/stable-diffusion-webui/scripts/xyz_grid.py", line 365, in draw_xyz_grid
    grid = images.image_grid(processed_result.images[start_index:end_index], rows=len(ys))
  File "/dfs/comicai/guanyu.zhao/stable-diffusion-webui/modules/images.py", line 55, in image_grid
    w, h = imgs[0].size
AttributeError: 'str' object has no attribute 'size'

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 a pull request may close this issue.

3 participants