Skip to content

Conversation

@bigcat88
Copy link
Contributor

@bigcat88 bigcat88 commented Dec 1, 2025

PR with price badges: Comfy-Org/ComfyUI_frontend#7077

API Node PR Checklist

Scope

  • Is API Node Change

Pricing & Billing

  • Need pricing update
  • No pricing update

If Need pricing update:

  • Metronome rate cards updated
  • Auto‑billing tests updated and passing

QA

  • QA done
  • QA not required

Comms

  • Informed Kosinkadink

@bigcat88 bigcat88 requested a review from Kosinkadink as a code owner December 1, 2025 17:27
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.

loosk good, merging

@Kosinkadink Kosinkadink merged commit 1cb7e22 into comfyanonymous:master Dec 2, 2025
11 checks passed
with av.open(path, mode='w', options={'movflags': 'use_metadata_tags'}) as output:
extra_kwargs = {}
if format != VideoContainer.AUTO:
extra_kwargs["format"] = format.value
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a breaking change, this doesn't work. The input is traditionally a string, not an enum value, eg from the SaveVideo node it emits "mp4" not VideoContainer.MP4.
This means the SaveVideo node does not work since this update.

Copy link
Contributor

Choose a reason for hiding this comment

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

cc @Kosinkadink @comfyanonymous fatal breaking bug for a lot of video workflows^

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this should fix custom nodes

#11046

Copy link
Contributor

Choose a reason for hiding this comment

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

to be clear for the record, the error replication does not require custom nodes at all, though I see you did notice and patch the native SaveVideo node in the PR, so that looks good to me

@bigcat88 bigcat88 deleted the feat/api-nodes/kling-video-o1 branch December 2, 2025 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants