search

WASasquatch / was-node-suite-comfyui

An extensive node suite for ComfyUI with over 210 new nodes
MIT License
1.15k stars 170 forks source link

Int to number node doesn't work #30

Closed lcolok closed 1 year ago

WASasquatch commented 1 year ago

See: https://github.com/comfyanonymous/ComfyUI/issues/368#issuecomment-1498003825

Maybe voice your opinion on how this is considered a bug to randomly not allow certain types as input only.

WASasquatch commented 1 year ago

These are removed until the issue is resolved.

lcolok commented 1 year ago

These are removed until the issue is resolved.

This is a pity, because I was hoping to build a workflow that could read the size of incoming images and match them with corresponding empty latents for image generation and other operations.

WASasquatch commented 1 year ago

These are removed until the issue is resolved.

This is a pity, because I was hoping to build a workflow that could read the size of incoming images and match them with corresponding empty latents for image generation and other operations.

Well there is no node to get the width/height of a image and pass it along, but the rest of the functionality is already there if it were, such as Number to Int to then plug into a EmptyLatent. Wouldn't be a hard node to make.

WASasquatch commented 1 year ago

I added a Image Size to Number node in 2ff0c04a656d1d69f8eb24d76159dfd8cbf59a04

I'll add a latent one later.

lcolok commented 1 year ago

I added a Image Size to Number node in 2ff0c04

I'll add a latent one later.

That's really helpful for my workflow! I appreciate your efforts and I'm looking forward to seeing the latent one as well.