Skip to content

added camera FPS settings to recording and inference instead of hardcoded 30fps, naming - #107

Closed
bellabzhu wants to merge 6 commits into
huggingface:mainfrom
bellabzhu:dev
Closed

bellabzhu wants to merge 6 commits into
huggingface:mainfrom
bellabzhu:dev

Conversation

@bellabzhu

Copy link
Copy Markdown

No description provided.

bellabzhu and others added 6 commits September 11, 2026 13:33
The inference screen hard-coded camera framerate to DEFAULT_FPS (30)
when building cameraDict. Add an editable "Camera FPS" number field,
defaulting to DEFAULT_FPS, mirroring the existing "Max duration" input.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
feat(frontend): add editable Camera FPS field to inference modal
Rename the top bar label and the page title/OG tags from "LeLab" to
"LeLab-Bellab" so it's always visually obvious which build (original
vs. this fork) is running in the browser, since both serve on the same
localhost:8000 origin.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
feat(frontend): brand landing header and tab as LeLab-Bellab
The recording request hard-coded fps: 30 for the dataset's overall
sampling rate, separate from (and often mismatched with) each
camera's own configured fps. lerobot's sanity check compares dataset
fps against the robot's actual camera fps and errors on a mismatch —
e.g. cameras configured at 15fps but the dataset requesting 30fps.

Add an editable "Recording FPS" field to the recording modal,
defaulting to the first configured camera's own fps (falling back to
30 if none is set), and use it instead of the hard-coded constant.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
fix(frontend): stop hard-coding recording fps to 30
@bellabzhu
bellabzhu marked this pull request as draft September 11, 2026 16:32
@bellabzhu bellabzhu closed this Sep 11, 2026
@bellabzhu

Copy link
Copy Markdown
Author

Sorry, this pull request was a mistake. If you can delete it, I would appreciate.

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.

1 participant