Skip to content

feat(schema): ego_pose.imu and gravity-free acceleration - #321

Merged
jacoblambert merged 2 commits into
v1-devfrom
v1/ego-imu
Oct 28, 2026
Merged

jacoblambert merged 2 commits into
v1-devfrom
v1/ego-imu

Conversation

@jacoblambert

@jacoblambert jacoblambert commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Planning review of the backfilled 1.0 samples found that ego_pose.acceleration held the raw IMU sample (gravity included, with a bias of about +0.5 m/s² at rest), while their planner uses Autoware's gravity-free estimate (/localization/acceleration, RMSE 0.12 m/s² against d(speed)/dt).

Both are useful for planning team, so we add support for both in this PR.

  • ego_pose.acceleration: (ax, ay, az) in the vehicle frame, without gravity: the localization estimate, what Autoware feeds the planner. This reverses the "gravity included" line from feat(schema): caller-given metadata in V0Migrator.migrate (backfill) #319.
  • New ego_pose.imu: option[[float;6]] = (ax, ay, az, wx, wy, wz), the raw IMU in the vehicle frame: specific force with gravity (az ≈ +9.8 m/s² at rest) and angular velocity.
  • Spec sentence: an ego_pose sits at a sample time; twist, acceleration, imu, geocoordinate and covariance hold the latest message at or before timestamp, best effort; the pose itself is interpolated.

0.x ego_pose.acceleration is dropped when loading or migrating a 0.x dataset. Checked on local datasets: where 0.x has it (22a86de5, 77f0c093), it is the raw Tamagawa IMU in FRD (az ≈ −9.8), so it is neither 1.0 field, and the IMU setup is not recorded in the dataset. V0Migrator writes a note; the backfill restores both fields from the bag.

Master sheet: ego_pose.imu added and acceleration reworded as proposed.

Tests

  • imu with 3 values is rejected (test_invalid_record).
  • A 0.x dataset with acceleration migrates with both fields null and the note.
  • The sample dataset gets imu: null; its manifest hash is updated.

@jacoblambert
jacoblambert requested a review from ktro2828 October 26, 2026 02:10

@ktro2828 ktro2828 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jacoblambert
jacoblambert merged commit cd4d7be into v1-dev Oct 28, 2026
4 of 5 checks passed
@jacoblambert
jacoblambert deleted the v1/ego-imu branch October 28, 2026 01:19
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