Skip to content

Support creating episodes from event IDs - #167

Merged
Mo-Elshenawy1 merged 3 commits into
mainfrom
claude/keen-ritchie-04t8dv
Oct 14, 2026
Merged

Mo-Elshenawy1 merged 3 commits into
mainfrom
claude/keen-ritchie-04t8dv

Conversation

@Mo-Elshenawy1

Copy link
Copy Markdown
Contributor

Changelog

Client.create_episodes accepts event_id to create an episode from an event.

Docs

None

Description

An episode input may now specify event_id instead of recordings; the API resolves the event's time range and overlapping recordings. event_id cannot be combined with recordings, start_time, or end_time. Recording-based inputs are unchanged. Covered by new unit tests; examples/datasets.py shows usage.

BeforeAfter

create_episodes(project_id=…, episodes=[{"event_id": "evt_…"}])
→ KeyError: 'recordings'

create_episodes(project_id=…, episodes=[{"event_id": "evt_…"}])
→ [{"id": "ep_…", "created": True}]

🤖 Generated with Claude Code

https://claude.ai/code/session_01B8ez4Z8Wq7dTA2sbcm76V2


Generated by Claude Code

`create_episodes` accepts `event_id` as an alternative to `recordings`.
The API resolves the event's time range and recordings; `event_id`
cannot be combined with `recordings`, `start_time`, or `end_time`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01B8ez4Z8Wq7dTA2sbcm76V2
claude[bot]

This comment was marked as outdated.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01B8ez4Z8Wq7dTA2sbcm76V2

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Comment thread foxglove/client/api.py Outdated
Co-Authored-By: mguida22 <mguida22@users.noreply.github.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01B8ez4Z8Wq7dTA2sbcm76V2

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Mo-Elshenawy1
Mo-Elshenawy1 merged commit b627f23 into main Oct 14, 2026
2 checks passed
@Mo-Elshenawy1
Mo-Elshenawy1 deleted the claude/keen-ritchie-04t8dv branch October 14, 2026 23:19
@linear-code

linear-code Bot commented Oct 14, 2026

Copy link
Copy Markdown

DC-1836

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants