[API Nodes] made Reve node price badges more precise#13154
[API Nodes] made Reve node price badges more precise#13154Kosinkadink merged 1 commit intomasterfrom
Conversation
Signed-off-by: bigcat88 <bigcat88@icloud.com>
📝 WalkthroughWalkthroughUpdated the 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ 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. Comment |
There was a problem hiding this comment.
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_reve.py`:
- Around line 246-255: The current ternary uses $isFast before $enabled so fast
variants always return 0.01001 and never get enabled-upscale pricing; update the
expression around $isFast, $enabled and $factor so when $isFast is true you
still branch on $enabled and $factor (e.g. $isFast ? ($enabled ? {fast-enabled
USD rates by $factor} : {"type":"usd","usd":0.01001,"format":$fmt}) : ($enabled
? {regular-enabled USD rates by $factor} :
{"type":"usd","usd":0.0572,"format":$fmt})), ensuring $isFast, $enabled and
$factor are referenced exactly as in the diff.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 87d5d5c0-b7b9-4e44-98c0-3d2b83a1fc83
📒 Files selected for processing (1)
comfy_api_nodes/nodes_reve.py
Added price accounting for the
upscalerwidget parameter.The provider doesn't have any official, precise documentation on this, so I simply looked at the price and adjusted it based on testing.
Click to see images with badges
API Node PR Checklist
Scope
Pricing & Billing
If Need pricing update:
QA
Comms