search

yolain / ComfyUI-Easy-Use

In order to make it easier to use the ComfyUI, I have made some optimizations and integrations to some commonly used nodes.
GNU General Public License v3.0
1.03k stars 71 forks source link

For loop bug #487

Open SmartRick opened 2 weeks ago

SmartRick commented 2 weeks ago

The for loop has A cache problem. Flow A executes normally for the first time and is switched to flow B. The number of loops is still the number of loops of flow A. You need to restart the for loop to solve the problem

yolain commented 2 weeks ago

Are you using the latest version?

Shyryp commented 2 days ago

I have the same problem, and it is not entirely clear why this happens - it is a completely random error, the steps to which I have not yet been able to guess.

If there are 2 cycles for the first run, then there may also be 2 cycles for the next run, although total was set to 1 or any other number other than 2.

I have updated to the latest version of ComfyUI and Easy Use nodes.

yolain commented 1 day ago

I have the same problem, and it is not entirely clear why this happens - it is a completely random error, the steps to which I have not yet been able to guess.

If there are 2 cycles for the first run, then there may also be 2 cycles for the next run, although total was set to 1 or any other number other than 2.

I have updated to the latest version of ComfyUI and Easy Use nodes.

I found a new way to get the totals correctly, the original way may have been problematic. I'm working on an update, can try it on the next commit

Shyryp commented 14 hours ago

I assume the issue can be closed, as the issue is currently not occurring.

@SmartRick, check that everything is working for you and close the Issue if everything is ok.