Skip to content

fix: modify replay_npz.py compatibility with MotionLoader API#4

Open
masahiro-kubota wants to merge 1 commit into
BoosterRobotics:mainfrom
masahiro-kubota:codex/fix-replay-npz-motionloader
Open

fix: modify replay_npz.py compatibility with MotionLoader API#4
masahiro-kubota wants to merge 1 commit into
BoosterRobotics:mainfrom
masahiro-kubota:codex/fix-replay-npz-motionloader

Conversation

@masahiro-kubota

@masahiro-kubota masahiro-kubota commented May 16, 2026

Copy link
Copy Markdown

Description

replay_npz.py was not compatible with the current MotionLoader API, so the replay script could not run.

Traceback (most recent call last):
  File "/mnt/ssd2/booster/booster_train/scripts/replay_npz.py", line 148, in <module>
    main()
  File "/mnt/ssd2/booster/booster_train/scripts/replay_npz.py", line 143, in main
    run_simulator(sim, scene)
  File "/mnt/ssd2/booster/booster_train/scripts/replay_npz.py", line 104, in run_simulator
    motion = MotionLoader(
             ^^^^^^^^^^^^^
TypeError: MotionLoader.__init__() missing 1 required positional argument: 'track_joint_names'

This PR updates replay_npz.py to pass the tracked body names and joint names expected by MotionLoader.

Test Performed

I verified locally that the reference motion can be replayed with:

python scripts/replay_npz.py \
  --motion ../booster_assets/motions/K1/k1_fight_001.npz \
  --device cuda:0
image

@masahiro-kubota masahiro-kubota changed the title [codex] Fix replay_npz compatibility with MotionLoader API Fix replay_npz compatibility with MotionLoader API May 16, 2026
@masahiro-kubota masahiro-kubota changed the title Fix replay_npz compatibility with MotionLoader API fix: modify replay_npz.py compatibility with MotionLoader API May 16, 2026
@masahiro-kubota masahiro-kubota force-pushed the codex/fix-replay-npz-motionloader branch 3 times, most recently from c7376df to de92724 Compare May 16, 2026 15:42
@masahiro-kubota masahiro-kubota force-pushed the codex/fix-replay-npz-motionloader branch from de92724 to 540d31f Compare May 16, 2026 15:45
@masahiro-kubota masahiro-kubota marked this pull request as ready for review May 16, 2026 15:50
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