Skip to content

Remove odometry data from mobile robot state#64

Merged
shantanuparab-tr merged 1 commit into
trossen-aifrom
02-13-remove_odometry_data_from_mobile
Feb 20, 2026
Merged

Remove odometry data from mobile robot state#64
shantanuparab-tr merged 1 commit into
trossen-aifrom
02-13-remove_odometry_data_from_mobile

Conversation

@shantanuparab-tr

Copy link
Copy Markdown
Collaborator

Summary

Removed odometry position data (x, y, theta) from the Trossen AI Mobile robot state representation, retaining only velocity-based information.


Changes

  • Removed "odom_x", "odom_y", and "odom_theta" from state_names in the motor_features property
  • Removed corresponding odometry readings from the read_base method
  • Simplified state representation to include only "linear_vel" and "angular_vel" in addition to arm motor names

Motivation

This change was made to:

  • Avoid issues related to the odometry mapping bug that caused inconsistencies in state representation
  • Standardize the dataset format across all mobile platforms by using a consistent velocity-based base state

Validation

The change was validated by:

  • Replaying both recorded actions and observations from a collected dataset
  • Verifying that the robot exhibited identical behavior after the modification

This confirms that the action space remains consistent with the observation space and that removing absolute odometry values does not alter functional behavior.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Trossen AI Mobile robot’s state representation to remove absolute odometry (x, y, theta) and keep only velocity-based base state, aligning mobile platform datasets around a consistent base signal.

Changes:

  • Removed odom_x, odom_y, odom_theta from motor_features()["observation.state"].names.
  • Removed odometry fields from get_base_state() so it returns only linear_vel and angular_vel.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lerobot/common/robot_devices/robots/trossen_ai_mobile.py
@shantanuparab-tr

Copy link
Copy Markdown
Collaborator Author

@copilot open a new pull request to apply changes based on the comments in this thread

Copilot AI commented Feb 20, 2026

Copy link
Copy Markdown

@shantanuparab-tr I've opened a new pull request, #67, to work on those changes. Once the pull request is ready, I'll request review from you.

@shantanuparab-tr
shantanuparab-tr merged commit f632c88 into trossen-ai Feb 20, 2026
6 checks passed
@shantanuparab-tr
shantanuparab-tr deleted the 02-13-remove_odometry_data_from_mobile branch February 20, 2026 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants