feat(api-nodes-xAI): updated models and pricing#12756
Conversation
1ed372f to
59901d7
Compare
📝 WalkthroughWalkthroughThis pull request modifies Grok API node implementations to add support for expanded parameters and multi-image handling. Changes include a new 🚥 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_grok.py`:
- Around line 244-252: The current branching allows up to 3 images for any
non-pro model which inadvertently includes "grok-imagine-image-beta"; change the
second condition so the 3-image limit only applies when model ==
"grok-imagine-image" (i.e. replace the elif guard with something like: elif
model == "grok-imagine-image" and get_number_of_images(image) > 3: raise
ValueError("A maximum of 3 input images is supported.")); keep the existing
single-image pro check (model == "grok-imagine-image-pro") and leave the
aspect_ratio/single-image validation as-is.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: c2c7ab37-e89f-4b5b-821a-dced5a65a321
⛔ Files ignored due to path filters (1)
comfy_api_nodes/apis/grok.pyis excluded by!comfy_api_nodes/apis/**
📒 Files selected for processing (1)
comfy_api_nodes/nodes_grok.py
Summary of changes:
grok-imagine-image-proto image generation and image edit nodesgrok-imagine-imageandgrok-imagine-video(non-beta) to all model dropdowns2Kresolution for Image generation/edit nodesgrok-imagine-imageis usedaspect_ratiowidget to GrokImageEdit nodeAPI Node PR Checklist
Scope
Pricing & Billing
If Need pricing update:
QA
Comms