Added new additional params for videoInference support#289
Added new additional params for videoInference support#289Sirsho1997 wants to merge 2 commits intomainfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. To trigger a review, include ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches✨ Simplify code
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Comment |
There was a problem hiding this comment.
Pull request overview
This PR updates the SDK’s type definitions to support additional video-inference-related request parameters, primarily by extending ISettings and making IVideoInference.speech easier to provide as a plain dict (auto-coerced to the proper dataclass).
Changes:
- Added
voicePromptandsafetyFilterfields toISettings. - Updated
IVideoInference.speechto acceptdictinputs and coerces them toIVideoSpeechSettingsin__post_init__.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| style: Optional[str] = None | ||
| voicePrompt: Optional[str] = None | ||
| safetyFilter: Optional[bool] = None | ||
| thinking: Optional[str] = None |
| if self.speech is not None and isinstance(self.speech, dict): | ||
| self.speech = IVideoSpeechSettings(**self.speech) |
| providerSettings: Optional[VideoProviderSettings] = None | ||
| speech: Optional[IVideoSpeechSettings] = None | ||
| speech: Optional[Union[IVideoSpeechSettings, Dict[str, Any]]] = None | ||
| webhookURL: Optional[str] = None |
ISettingsnow includes:voicePrompt: Optional[str]safetyFilter: Optional[bool]