Add new model blueprints#14506
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughSix blueprint JSON files are added or updated. Five new blueprints are introduced: "Image Depth Estimation (Depth Anything 3)" wires 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@blueprints/Image` to Gaussian Splat (TripoSplat).json:
- Around line 354-362: The blueprint contains inconsistent link metadata where
the splat output at lines 354-362 has a duplicate link ID (156 appears twice in
the linkIds array), and node 109's IMAGE output references non-existent links
(166 and 194). Fix these issues by removing the duplicate 156 from the splat
output's linkIds array (keep only one instance) and replacing the invalid link
references in node 109's IMAGE output with correct link IDs that actually exist
in the blueprint's links array. Apply the same cleanup at lines 1024-1033 to
ensure all output linkIds arrays contain only valid, non-duplicate link
references that correspond to actual entries in the blueprint.
- Line 187: The exposed seed input is currently only wired to the KSampler node,
but the VAEDecodeTripoSplat node's seed input remains unconnected, causing it to
use the default hidden widget value instead of the user-provided seed. Create a
new link connection object that wires the seed output to the VAEDecodeTripoSplat
node's seed input by incrementing the lastLinkId counter and adding an
appropriate link entry in the links array. This wiring fix needs to be applied
at all affected locations in the file: the main location around line 187 where
lastLinkId is updated, and also at the corresponding locations around lines
253-258, 956-964, and 1498-1504 where similar VAEDecodeTripoSplat nodes with
unconnected seed inputs exist.
In `@blueprints/Text` to Image (Anima Base 1.0).json:
- Line 910: Fix the typo in the group title field that currently reads "Image
Size(1MP))". Remove the extra closing parenthesis at the end and add a space
between "Size" and the opening parenthesis to correct the formatting to "Image
Size (1MP)".
🪄 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: ab8fd127-8beb-43ea-9ab5-3c4e9e7192e6
📒 Files selected for processing (10)
blueprints/Character Replacement (SCAIL-2 Base).jsonblueprints/Character Replacement (SCAIL-2 Extend).jsonblueprints/Image Depth Estimation (Depth Anything 3).jsonblueprints/Image Edit (Bernini-R).jsonblueprints/Image to Gaussian Splat (TripoSplat).jsonblueprints/Text to Image (Anima Base 1.0).jsonblueprints/Text to Image (Anima).jsonblueprints/Text to Image (Ideogram v4).jsonblueprints/Video Depth Estimation (Depth Anything 3).jsonblueprints/Video Edit (Bernini-R).json
|
@alexisrolland need you to approve again, fixed an issue that was reported by the coderabbit |
Add model blueprints