Launch Speechify Simba models - #404
Conversation
Promote simba-3.2 and simba-3.0 from early access to active now that the provider key is fixed.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughUpdated the Speechify TTS Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
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 current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@runner/src/coval_bench/registries/models.py`:
- Around line 745-748: Add the existing _MULTI tag to the tags tuple for the
simba-3.2 model registry entry, preserving all current tags and status.
🪄 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: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 162881b0-a4e3-4f58-8826-1368f2200d65
📒 Files selected for processing (1)
runner/src/coval_bench/registries/models.py
| model="simba-3.2", | ||
| voice="geffen_32", | ||
| tags=(_STREAMING, _CLONE, _EMOTION), | ||
| status=_EARLY_ACCESS, | ||
| status=_ACTIVE, |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
Add the _MULTI tag to simba-3.2.
The PR objective requires Simba 3.2 to be marked multilingual, but its registry tags currently omit _MULTI. Because these tags are exposed as provider API facets, clients will not see or be able to filter this model as multilingual.
Proposed fix
voice="geffen_32",
- tags=(_STREAMING, _CLONE, _EMOTION),
+ tags=(_STREAMING, _MULTI, _CLONE, _EMOTION),
status=_ACTIVE,📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| model="simba-3.2", | |
| voice="geffen_32", | |
| tags=(_STREAMING, _CLONE, _EMOTION), | |
| status=_EARLY_ACCESS, | |
| status=_ACTIVE, | |
| model="simba-3.2", | |
| voice="geffen_32", | |
| tags=(_STREAMING, _MULTI, _CLONE, _EMOTION), | |
| status=_ACTIVE, |
🤖 Prompt for 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.
In `@runner/src/coval_bench/registries/models.py` around lines 745 - 748, Add the
existing _MULTI tag to the tags tuple for the simba-3.2 model registry entry,
preserving all current tags and status.
Promote simba-3.2 and simba-3.0 from early access to active now that the provider key is fixed.