was-node-suite-comfyui icon indicating copy to clipboard operation
was-node-suite-comfyui copied to clipboard

Two "Load Image Batch" in the same workflow leads to a bug

Open Zeelyne opened this issue 1 year ago • 3 comments

Hello,

I want to combine two lists of masks from two separate folders. I tried to initiate a batch process with the following workflow:

image

Interestingly, when I choose "random" or "single_image" in the "Load Image Batch," it works. However, if I choose "incremental_image," both "Load Image Batch" nodes keep outputing only the first image of their respective folders. No error message but the output is not what I suppose to get.

Then, I bypassed one of the "Load Image Batch" nodes to see the output when using "incremental_image." This time, it worked correctly! image

It seems that there is a bug when using two "Load Image Batch" nodes in the same workflow. Can anyone tell me how to solve this problem?

Many thanks.

Zeelyne avatar May 11 '24 08:05 Zeelyne

Oh I just found that, by using two different values in the "lable" for these two "load image batch", this problem can be solved. However I am not very clear with the behind reason for that. Waiting for a more detailed explanation for this issue.

Zeelyne avatar May 11 '24 08:05 Zeelyne

Oh I just found that, by using two different values in the "lable" for these two "load image batch", this problem can be solved. However I am not very clear with the behind reason for that. Waiting for a more detailed explanation for this issue.

The nodes are from before node classes were persistent, so it's data is being stored in a dictionary, and the key to that nodes data is the label.

These days, it could be updated to store it's data on the class itself, since they are persistent now, and just use the node's ID and have no need for a label.

WAS-PlaiLabs avatar May 14 '24 19:05 WAS-PlaiLabs

OK thank you! Hope to see the update

Zeelyne avatar May 16 '24 08:05 Zeelyne