[release/0.2.1] Change the prompt for the finetuning#722
Conversation
There was a problem hiding this comment.
Review Summary
This PR simplifies the language instruction prompt from a detailed description to a more concise task statement across 3 configuration files and documentation.
Changes Reviewed
| File | Change |
|---|---|
docs/pages/example_workflows/static_apple/step_3_policy_training.rst |
Updated documentation prompt |
isaaclab_arena_gr00t/lerobot/config/g1_static_apple_config.yaml |
Updated training config prompt |
isaaclab_arena_gr00t/policy/config/g1_static_apple_gr00t_closedloop_config.yaml |
Updated inference config prompt |
Observations
✅ Consistency: The prompt change is applied consistently across all relevant files (documentation, LeRobot training config, and closed-loop policy config).
✅ Commit hygiene: Good to see the follow-up commit removing the trailing period for cleaner prompt formatting.
Clarification Request
The prompt is changing from:
"Pick up the apple from the shelf and place it onto the plate on the same shelf next to it."
to:
"move the apple to the plate"
Question: Could you confirm this matches the prompt used during the finetuning of checkpoint-20000? Language-conditioned policies can be sensitive to prompt wording, so ensuring the inference prompt matches the training prompt is important for optimal performance.
This is an automated review. Please address any questions before merging.
23c7a78 to
7080605
Compare
|
LGTM 🚀 |
Summary
Align static apple prompt
Detailed description
move the apple to the plate.