ComfyUI
ComfyUI copied to clipboard
comfyanonymous
Silversith Main Branch
@comfyanonymous I don't expect you to merge this in, but I just thought I'd ask your opinion 😊
I gave this a try and unfortunately it crashed my browser probably because I have over 10k images in the output folder.
I gave this a try and unfortunately it crashed my browser probably because I have over 10k images in the output folder.
😅 I didn't think of that. I guess I'll have to cap the images to 100 thumbnails... Maybe less depending on the size of the images. 50 maybe?
I gave this a try and unfortunately it crashed my browser probably because I have over 10k images in the output folder.
😅 I didn't think of that. I guess I'll have to cap the images to 100 thumbnails... Maybe less depending on the size of the images. 50 maybe?
I gave this a try and unfortunately it crashed my browser probably because I have over 10k images in the output folder.
😅 I didn't think of that. I guess I'll have to cap the images to 100 thumbnails... Maybe less depending on the size of the images. 50 maybe?
Actually maybe I can disable the load from output if there's more than 50 images and only load for the session. That would stop the refresh load though, but maybe that's a better solution.
I gave this a try and unfortunately it crashed my browser probably because I have over 10k images in the output folder.
😅 I didn't think of that. I guess I'll have to cap the images to 100 thumbnails... Maybe less depending on the size of the images. 50 maybe?
I gave this a try and unfortunately it crashed my browser probably because I have over 10k images in the output folder.
😅 I didn't think of that. I guess I'll have to cap the images to 100 thumbnails... Maybe less depending on the size of the images. 50 maybe?
Actually maybe I can disable the load from output if there's more than 50 images and only load for the session. That would stop the refresh load though, but maybe that's a better solution.
@comfyanonymous I made some quick changes that I also only very quickly tested. It won't load the images form the output folder if there's more than 50 and will also then disable the delete all button
I gave this a try and unfortunately it crashed my browser probably because I have over 10k images in the output folder.
😅 I didn't think of that. I guess I'll have to cap the images to 100 thumbnails... Maybe less depending on the size of the images. 50 maybe?
I gave this a try and unfortunately it crashed my browser probably because I have over 10k images in the output folder.
😅 I didn't think of that. I guess I'll have to cap the images to 100 thumbnails... Maybe less depending on the size of the images. 50 maybe?
Actually maybe I can disable the load from output if there's more than 50 images and only load for the session. That would stop the refresh load though, but maybe that's a better solution.
@comfyanonymous I made some quick changes that I also only very quickly tested. It won't load the images form the output folder if there's more than 50 and will also then disable the delete all button
I'm considering changing the delete all (which deletes everything from the output folder) to only delete images that are displayed in the list
(Don't mind me, just cleaning up the PR backlog, and noticed this was I don't expect you to merge this in)
