fix: allow to process multiple images#96
Conversation
| # Redis configuration from environment variables | ||
| REDIS_HOST = os.environ.get("REDIS_HOST", "localhost") | ||
| REDIS_PORT = int(os.environ.get("REDIS_PORT", 6379)) | ||
| r = redis.Redis( |
| REDIS_HOST = os.environ.get("REDIS_HOST", "localhost") | ||
| REDIS_PORT = int(os.environ.get("REDIS_PORT", 6379)) | ||
| r = redis.Redis( | ||
| host='redis-13524.fcrce180.us-east-1-1.ec2.redns.redis-cloud.com', |
|
@bluematter thanks for putting the effort into this PR. it looks to me that you not only added support for multiple images, but also a lot more (like redis and loras and more). would you mind to clarify everything you have done here, so that we can do a proper review? |
|
Hey guys, thank you all for the great work. @bluematter wouldn't it be easier to just fix the batch loading separately and then add the other features sequentially? |
|
@bluematter @Andergraw @khengyun we have added support for multiple output images in #118, is that what you wanted? because multiple input-images are already possible |
This exactly what I wanted, thanks! How is it done? Using the batch input from Empty Latent node? Can It be done with runsync endpoint? |
|
Hi @bluematter — thanks for sending this, but this PR is unfortunately too far out of date to merge as-is. It modifies What the PR appears to add (handling multiple input images) is actually already supported on Two options:
Either way, the existing diff isn't recoverable into a modern PR. Sorry for the slow reply. |
Motivation
I want to pass in a number of outputs into comfy and get the same amount of outputs sent back in the API
Issues closed
#95