Skip to content

feat: sync go1 motrix ppo/sac and g1 motrix ppo tuning, fix related bugs - #48

Merged
TATP-233 merged 5 commits into
mainfrom
feat/motrix-go1-g1-sync
Mar 19, 2026
Merged

TATP-233 merged 5 commits into
mainfrom
feat/motrix-go1-g1-sync

Conversation

@Jaui7

@Jaui7 Jaui7 commented Mar 18, 2026

Copy link
Copy Markdown

Summary

  • sync go1 motrix ppo tuning
  • sync go1 motrix sac tuning
  • sync g1 motrix ppo tuning
  • fix related motrix train/play bugs

Fixes

  • fix go1 sac motrix collector obs mismatch in offpolicy train path
  • fix go1 sac motrix play_only reset unpack issue
  • fix g1 sac motrix config/runtime issues needed to start training
  • fix g1 sac motrix play_only obs-dim mismatch when loading checkpoints

Validation

  • verified go1 + sac + motrix training path
  • verified go1 + sac + motrix play_only path
  • verified g1 + sac + motrix smoke training path
  • verified g1 + sac + motrix play_only path

if len(reset_result) == 3:
_, obs_out, _ = reset_result
return obs_out
raise ValueError(f"Unexpected env.reset return format: {type(reset_result)!r}")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

这部分不做向前兼容了

Comment thread scripts/train_rsl_rl.py Outdated

_GO1_LEGACY_COMMAND_VEL_LIMIT = [[0.5, 0.0, 0.0], [0.5, 0.0, 0.0]]
_GO1_LEGACY_REWARD_CONFIG = {
"scales": {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

统一走 hydra 配置?

@TATP-233
TATP-233 merged commit aafa923 into main Mar 19, 2026
3 checks passed
@TATP-233
TATP-233 deleted the feat/motrix-go1-g1-sync branch March 19, 2026 05:45
TATP-233 added a commit that referenced this pull request May 4, 2026
feat: sync go1 motrix ppo/sac and g1 motrix ppo tuning, fix related bugs
TATP-233 added a commit that referenced this pull request Sep 4, 2026
feat: sync go1 motrix ppo/sac and g1 motrix ppo tuning, fix related bugs
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.

2 participants