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

IMG2IMG TIF batch fix img2img.py #12120

Merged
merged 1 commit into from
Jul 29, 2023

Conversation

DiabolicDiabetic
Copy link
Contributor

@DiabolicDiabetic DiabolicDiabetic commented Jul 28, 2023

[Bug]: IMG2IMG Batch will not locate TIF images #12106

Description

  • a simple description of what you're trying to accomplish
    get img2img batching to process tifs again

  • a summary of changes in code
    Adjusted line 20 to include .tif and .tiff

  • which issues it fixes, if any:
    In my tests, it is working, now

Checklist:

  • I have read contributing wiki page
  • I have performed a self-review of my own code
  • My code follows the style guidelines
  • My code passes tests
    The last instruction on the wiki/Tests says "Once the server is running, you can run tests with just py.test."
    @akx How do I do that? Console? From within the WebUI? Please provide clearer instructions for me to be able to confidently say I've run the test for pull request submission. I understand installing via the requirements.txt and added the argument to my command_args with the launcher, but otherwise I don't know what to do from there.

IMG2IMG batch tab wouldn't process tif images
@DiabolicDiabetic
Copy link
Contributor Author

Ahhhhh, scratch that - I understand, now. Run your own test series with the tool? If that's the case, my single line change I couldn't find any hiccups. Editing my original post to show testing complete

@w-e-w
Copy link
Collaborator

w-e-w commented Jul 28, 2023

by test we meant that, a simple automated test using API
image

you should also run your own other tests obviously

@AUTOMATIC1111 AUTOMATIC1111 merged commit fc16321 into AUTOMATIC1111:dev Jul 29, 2023
@w-e-w w-e-w mentioned this pull request Aug 24, 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.

3 participants