Skip to content

configs: declare proprio and actions dims once per HPT model config - #634

Merged
RyanPCo merged 1 commit into
mainfrom
ryanco/phase3-model-dims
Oct 19, 2026
Merged

RyanPCo merged 1 commit into
mainfrom
ryanco/phase3-model-dims

Conversation

@RyanPCo

@RyanPCo RyanPCo commented Sep 13, 2026 •

Copy link
Copy Markdown
Contributor

stem input_dim, infer_ac_dims, act_dim and enc-dec output_dim now interpolate
robomimic_model.dims.{.proprio,.action,action_width}. Resolved values
are unchanged (pinned by tests/unit/test_model_dims.py). Pi configs untouched.

Co-Authored-By: Claude Fable 5.1 [email protected]
Claude-Session: https://claude.ai/code/session_01MWtR1x5tjREwoxQZAnUGrh

🤖 Generated with Claude Code

https://claude.ai/code/session_01MWtR1x5tjREwoxQZAnUGrh

@RyanPCo
RyanPCo force-pushed the ryanco/phase3-model-dims branch 2 times, most recently from b0fe5ea to 0df1305 Compare September 15, 2026 13:31
@RyanPCo
RyanPCo force-pushed the ryanco/phase2-norm-cache branch from 27eb832 to 73e2073 Compare September 15, 2026 13:31
@RyanPCo
RyanPCo force-pushed the ryanco/phase3-model-dims branch from 0df1305 to fb14ce8 Compare September 15, 2026 15:04
@RyanPCo
RyanPCo force-pushed the ryanco/phase2-norm-cache branch from 73e2073 to 3157eb9 Compare September 15, 2026 15:04
@RyanPCo RyanPCo changed the title configs: declare per-embodiment dims once per HPT model config configs: declare proprio and actions dims once per HPT model config Sep 15, 2026
@RyanPCo
RyanPCo force-pushed the ryanco/phase3-model-dims branch from fb14ce8 to 35502e3 Compare September 15, 2026 15:11
@RyanPCo
RyanPCo marked this pull request as ready for review September 15, 2026 15:16
stem input_dim, infer_ac_dims, act_dim and enc-dec output_dim now interpolate
robomimic_model.dims.{<emb>.proprio,<emb>.action,action_width}. Resolved values
are unchanged (pinned by tests/unit/test_model_dims.py). Pi configs untouched.

Co-Authored-By: Claude Fable 5.1 <[email protected]>
Claude-Session: https://claude.ai/code/session_01MWtR1x5tjREwoxQZAnUGrh
Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
Claude-Session: https://claude.ai/code/session_01BbQNNaMAbTNMypegmJkv4s

RyanPCo commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor Author

Merge activity

  • Oct 19, 7:23 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Oct 19, 7:23 PM UTC: @RyanPCo merged this pull request with Graphite.

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