Skip to content

Add session video dimensions options - #56

Merged
sebvanleuven merged 2 commits into
mainfrom
codex/session-video-dimensions
Jul 21, 2026
Merged

Add session video dimensions options#56
sebvanleuven merged 2 commits into
mainfrom
codex/session-video-dimensions

Conversation

@bc-anam

@bc-anam bc-anam commented May 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add video_width and video_height to SessionOptions
  • serialize them as sessionOptions.videoWidth and sessionOptions.videoHeight
  • validate that dimensions are positive integers and provided together

Verification

  • uv run --extra dev pytest

Summary by cubic

Add support for optional session video dimensions. You can now set video_width and video_height in SessionOptions to request a specific output size.

  • New Features
    • Add video_width and video_height; must be provided together and be positive integers.
    • Serialize as videoWidth and videoHeight in the session payload.
    • Update README with a supported-resolutions table (Cara-3/Cara-4, landscape and Cara-4 portrait examples) and add tests for serialization and validation.

Written for commit 36f7cd9. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

Re-trigger cubic

bc-anam and others added 2 commits July 21, 2026 13:38
Document supported Cara-3/Cara-4 landscape and portrait sizes with
examples for both orientations, aligned with Agora session docs.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sebvanleuven
sebvanleuven force-pushed the codex/session-video-dimensions branch from dad709e to 36f7cd9 Compare July 21, 2026 13:07
@sebvanleuven
sebvanleuven self-requested a review July 21, 2026 13:07
@sebvanleuven
sebvanleuven merged commit 89f6d92 into main Jul 21, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants