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

processed image grid is not saved to output directory (left in temp) #10

Closed
jmp909 opened this issue Aug 1, 2023 · 1 comment · Fixed by #19
Closed

processed image grid is not saved to output directory (left in temp) #10

jmp909 opened this issue Aug 1, 2023 · 1 comment · Fixed by #19

Comments

@jmp909
Copy link

jmp909 commented Aug 1, 2023

Describe the bug
processed image grid is not saved to output directory .. remains in eg %AppData%\Local\Temp\Gradio

To Reproduce
Steps to reproduce the behavior:

  • run a generaton with a batch size of eg 4 images
  • Observe pre-processed grid is output to correct directory, but the processed grid remains in the temp gradio folder

Expected behavior
Post-process image grid should be output to relevant directory
(Note: ideally pre-processed image grid should only be output if "Keep original image before swapping" is checked)

Desktop (please complete the following information):

  • OS: Windows 11
  • Browser: Opera GX

thanks

@glucauze
Copy link
Owner

glucauze commented Aug 1, 2023

That's true, i do not save the grid in current version :

                # Generate grid :
                if opts.return_grid and len(images) > 1:
                    grid = imgutils.create_square_image(images)
                    text = processed.infotexts[0]
                    infotexts.insert(0, text)
                    if opts.enable_pnginfo:
                        grid.info["parameters"] = text
                    images.insert(0, grid)

Will try to fix that in next version.

@glucauze glucauze linked a pull request Aug 3, 2023 that will close this issue
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.

2 participants