Skip to content

[Partner Nodes] add 1080p resolution for SeeDance 2.0 model#13437

Merged
Kosinkadink merged 1 commit intomasterfrom
feat/api-nodes/seedance2-1080p
Apr 17, 2026
Merged

[Partner Nodes] add 1080p resolution for SeeDance 2.0 model#13437
Kosinkadink merged 1 commit intomasterfrom
feat/api-nodes/seedance2-1080p

Conversation

@bigcat88
Copy link
Copy Markdown
Contributor

@bigcat88 bigcat88 commented Apr 16, 2026

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

Signed-off-by: bigcat88 <bigcat88@icloud.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 16, 2026

📝 Walkthrough

Walkthrough

The pull request updates the Seedance 2.0 node implementations in comfy_api_nodes/nodes_bytedance.py. The _seedance2_text_inputs() and _seedance2_reference_inputs() functions now accept a resolutions parameter to support variable resolution options per model. The "Seedance 2.0" model now supports 480p, 720p, and 1080p resolutions, while "Seedance 2.0 Fast" supports 480p and 720p. Price badge expressions are updated to handle 1080p pricing with a new $rate1080 variable. Additionally, the long-polling configuration for Seedance 2.0 task execution was modified to set max_poll_attempts=180 alongside the existing polling interval.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding 1080p resolution support for the SeeDance 2.0 model, which is the primary focus of the PR modifications.
Description check ✅ Passed The pull request description clearly relates to the changeset by documenting completion of the API Node PR checklist and confirming QA was done.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 16, 2026

Caution

Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted.

Error details
{}

@Kosinkadink Kosinkadink merged commit d0c53c5 into master Apr 17, 2026
17 checks passed
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.

2 participants