hotfix: disable RTC inference engine for ACT checkpoints - #100
Closed
Mokuroh54 wants to merge 2 commits into
Closed
Conversation
RTC (Real-Time Chunking) regenerates action chunks through lerobot's flow-matching processor — a path ACT checkpoints don't have — so an ACT run started under RTC breaks at rollout. Both engine selectors (DeployPanel, InferenceModal) now force sync for ACT: the RTC option is disabled with an explanatory hint, and the submitted inference_engine is derived (isAct ? "sync" : selection) so a stale selection can never reach the backend. Mirrors the existing ACT-only gating of temporal ensembling. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…r-act # Conflicts: # frontend/src/components/landing/InferenceModal.tsx # frontend/src/components/studio/DeployPanel.tsx
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
RTC (Real-Time Chunking) regenerates action chunks through lerobot's flow-matching processor — a path ACT checkpoints don't have — so an ACT run started under RTC breaks at rollout.
Both engine selectors (studio
DeployPanel, landingInferenceModal) now force Sync whenever the selected checkpoint is ACT:SelectItemis disabled, with a new hint explaining why (en + zh-CN keys added).inference_engineis derived (isAct ? "sync" : selection), not a state reset, so a stale RTC selection from a previous checkpoint can never reach the backend — including across the policy-config fetch race.policy_type, and the backend default is alreadysync).Why into
mainHotfix for released code — RTC + ACT is selectable on
maintoday.sync_stagingwill carry it back intostagingautomatically.Testing
npx tsc --noEmit -p tsconfig.app.json/-p tsconfig.node.json— passnpm test— i18n catalog parity + dynamic-key tests pass; the one failing test (LanguageContext.test.tsx › switches back to English) fails identically on pristineorigin/main(pre-existing)npm run lint— 4 errors, all pre-existing in untouched files (baseline-identical)npm run build— pass (dist not committed; CI rebuilds it)🤖 Generated with Claude Code