ImageResizeKJv2 dead code and dubious conditionals

[set-soft]

In this node you have

ComfyUI-KJNodes/nodes/image_nodes.py

Lines 2508 to 2511 in b1fffd3

	if width == 0:
	width = W
	if height == 0:
	height = H

Which seems to be in conflict with:

ComfyUI-KJNodes/nodes/image_nodes.py

Lines 2518 to 2520 in b1fffd3

	if keep_proportion == "resize" or keep_proportion.startswith("pad") or pillarbox_blur:
	# If one of the dimensions is zero, calculate it to maintain the aspect ratio
	if width == 0 and height != 0:

How can width be 0 if it was assigned the image width?

I think the original intention was:

if width == 0 and height == 0:
    width, height = W, H

BTW: why did you remove the get_image_size input from the previous version, I think this is handy

[rugabunda]

Is this related? Recent updates to ImageResizeKJv2 broke the workflow

• ImageResizeKJv2 401:
  • Failed to convert an input value to a INT value: per_batch, Output: 1 x 608 x 730 | 5.08MB, invalid literal for int() with base 10: 'Output: 1 x 608 x 730 | 5.08MB'

EDIT:

Fixed by right clicking and fixing/rebuilding the node

[set-soft]

This not a real bug, just a dubious logic.
The node behaves as expected but a big chunk of code is just unreachable.
The logic of copying the width and height from the image is coherent with the rest, but then the code pretends that width and height might be 0, which is impossible.