Skip to content

Release 0 4 3#68

Open
lalit-videodb wants to merge 6 commits intomainfrom
release-0-4-3
Open

Release 0 4 3#68
lalit-videodb wants to merge 6 commits intomainfrom
release-0-4-3

Conversation

@lalit-videodb
Copy link
Collaborator

@lalit-videodb lalit-videodb commented Mar 17, 2026

Pull Request

Description:
Release 0.4.3 — Fixes type annotations, IDE resolution issues, and improves developer experience for clip(), Shot, and CaptionAsset APIs.

Changes:

  • Fix VideoAsset/AudioAsset start/end type from int to Optional[float] (asset.py)
  • Fix clip() return type from str to SearchResult — resolves unresolved .play() and .get_shots() in IDEs (video.py)
  • Add Literal type validation for clip() content_type and model_name params (video.py)
  • Pass stream_url/player_url from search API response to Shot objects (shot.py, search.py)
  • Add warnings.warn() for CaptionAsset(src='auto') when video indexing is required (editor.py)
  • Bump videodb-capture-bin dependency to >=0.2.10 (setup.py)
  • Bump version to 0.4.3 (about.py)

Related Issues:

  • Addresses ENG-161 — start parameter type mismatch (int vs float warning)
  • Addresses ENG-162 — model_name / content_type validation on clip()
  • Addresses ENG-163 — Unresolved play() and get_shots() on clip result
  • Addresses ENG-165 — stream_url and player_url are None in Shot object
  • Addresses ENG-166 — Stream not played after adding subtitles (CaptionAsset indexing warning)
  • Addresses ENG-167 — Editor 500 clips internal server error (server-side, not SDK)

Testing:

  • Verified VideoAsset(asset_id, start=1.5, end=3.0) no longer raises type warnings
  • Verified video.clip() returns SearchResult — IDE resolves .play() and .get_shots()
  • Verified clip() shows autocompletion for content_type ("spoken", "visual", "multimodal") and model_name ("basic", "pro", "ultra")
  • Verified Shot.stream_url is None from search (server doesn't send per-doc URLs), populated after compile() or shot.generate_stream()
  • Verified CaptionAsset(src='auto') emits a UserWarning about indexing requirement
  • ENG-167 (500 clips / internal server error) is a server-side limit — no SDK fix needed

- Fix VideoAsset/AudioAsset start/end type from int to Optional[float]
- Fix clip() return type from str to SearchResult
- Add Literal types for clip() content_type and model_name params
- Pass stream_url/player_url from search API response to Shot objects
- Add warnings.warn() for CaptionAsset(src='auto') indexing requirement
Copy link
Collaborator

@ankit-v2-3 ankit-v2-3 left a comment

Choose a reason for hiding this comment

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

Looks good to me

- Channels with `camera:` prefix now go to `channels.cameras` instead
  of `channels.displays`. Cameras excluded from `all()`.
- Switched audio grouping from name-based heuristics to channel ID
  prefix matching (`mic:`, `system_audio:`) for consistency with Node SDK.
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.

3 participants