Skip to content

[API Nodes] add new Topaz model#13175

Merged
Kosinkadink merged 1 commit intomasterfrom
feat/api-nodes/topaz-update
Mar 26, 2026
Merged

[API Nodes] add new Topaz model#13175
Kosinkadink merged 1 commit intomasterfrom
feat/api-nodes/topaz-update

Conversation

@bigcat88
Copy link
Copy Markdown
Contributor

@bigcat88 bigcat88 commented Mar 26, 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 Mar 26, 2026

📝 Walkthrough

Walkthrough

This change adds a new upscaler model entry to the UPSCALER_MODELS_MAP dictionary in comfy_api_nodes/nodes_topaz.py. The entry maps the display name "Starlight Precise 2.5" to the model identifier "slp-2.5". This addition expands the available upscaler options that can be selected through the TopazVideoEnhance node's upscaler_model input parameter. The modification consists of a single line addition with no deletions.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The PR title directly and specifically describes the main change: adding a new Topaz model to the upscaler options.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The pull request description is related to the changeset, which adds a new Topaz upscaler model to the API nodes. The checklist format is standard for API node changes.

✏️ 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.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@comfy_api_nodes/nodes_topaz.py`:
- Line 41: The mapping entry "Starlight Precise 2.5": "slp-2.5" in
comfy_api_nodes/nodes_topaz.py uses an identifier that isn't documented by
Topaz; confirm the correct model id with Topaz support and update or remove this
mapping to avoid runtime failures. Locate the models mapping in nodes_topaz.py
(the dictionary containing "Starlight Precise 2.5") and either replace "slp-2.5"
with the official API identifier provided by Topaz or remove/flag the model
entry and add a fallback/error when the user selects it so selection cannot
proceed with an invalid id. Ensure any tests or UI labels referencing "Starlight
Precise 2.5" use the updated identifier or reflect its unavailability.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c09a7ad7-c106-4be8-ba5e-f8ecbb60e33d

📥 Commits

Reviewing files that changed from the base of the PR and between b0fd65e and eb736a4.

📒 Files selected for processing (1)
  • comfy_api_nodes/nodes_topaz.py

@Kosinkadink Kosinkadink merged commit 8165485 into master Mar 26, 2026
17 checks passed
Kosinkadink pushed a commit that referenced this pull request Mar 26, 2026
Signed-off-by: bigcat88 <bigcat88@icloud.com>
comfyanonymous pushed a commit that referenced this pull request Mar 26, 2026
Signed-off-by: bigcat88 <bigcat88@icloud.com>
@bigcat88 bigcat88 deleted the feat/api-nodes/topaz-update branch March 27, 2026 11:22
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