Skip to content

Conversation

bigcat88
Copy link
Contributor

@bigcat88 bigcat88 commented Oct 6, 2025

The test workflow was saved in V1 nodes and loaded and tested in V3:

simplescreenrecorder-2025-10-06_18.26.31.mp4

Note:

  1. I was not able to get Camera Control working with most parameters in the V1 Nodes version. Error: camera_control.config is not allowed for this type: right_turn_forward.
  2. Lip Sync Audio wasn’t tested either because it rejects MP3 and WAV inputs. Error: "Audio format is invalid".

We can investigate these in a separate PR if needed, since this conversion PR does not change behavior or include fixes.

Objects git diff:

object_info_difference.zip

@bigcat88 bigcat88 requested a review from Kosinkadink as a code owner October 6, 2025 15:40
@bigcat88
Copy link
Contributor Author

bigcat88 commented Oct 6, 2025

+label: Core

@comfy-pr-bot comfy-pr-bot added the Core Core team dependency label Oct 6, 2025
Copy link
Collaborator

@Kosinkadink Kosinkadink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

object_info diff and code seems good, merging

@Kosinkadink Kosinkadink merged commit 8c19910 into comfyanonymous:master Oct 6, 2025
12 checks passed
@bigcat88 bigcat88 deleted the v3/nodes_kling branch October 7, 2025 05:49
toxicwind pushed a commit to toxicwind/ComfyUI that referenced this pull request Oct 12, 2025
gmaOCR pushed a commit to gmaOCR/ComfyUI that referenced this pull request Oct 14, 2025
adlerfaulkner pushed a commit to LucaLabsInc/ComfyUI that referenced this pull request Oct 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Core Core team dependency

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants