Skip to content

Expand zero gravity save data#78

Merged
K-Meech merged 18 commits into
mainfrom
km/zero-gravity-data
May 14, 2026
Merged

Expand zero gravity save data#78
K-Meech merged 18 commits into
mainfrom
km/zero-gravity-data

Conversation

@K-Meech
Copy link
Copy Markdown
Collaborator

@K-Meech K-Meech commented May 13, 2026

For #20

See extra info from David on the issue: #20 (comment)

  • adds additional save data for zero gravity
  • re-names image files to match pose names from spreadsheet
  • At the time of our last meeting (when David said we could remove all sway measurements), I had already finished refactoring TrackerBuffers so it could handle both head position (x/y/z) and angle (roll/pitch/yaw) data. Even though this is no longer needed for this PR I thought I'd leave it in as:
    • It gives us flexibility to calculate velocities in any direction / angle (in case David wants to add these sorts of measures in future)
    • It means we can now store head poses in one buffer, and calculate any of the associated position or angle velocities. This means that e.g. with RocketLaunch we no longer need to store the same head pose data in two separate buffers to get the pitch and yaw velocities - we can store the data once and calculate both directly from there.
    • See what you think - this refactoring can also be removed, if it's not useful
  • Save data docs are updated

@K-Meech K-Meech requested review from mmcleod89 and thompson318 May 13, 2026 09:55
Copy link
Copy Markdown
Member

@thompson318 thompson318 left a comment

Choose a reason for hiding this comment

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

This looks good to me. Following the same pattern as the other minigames so is easily comprehensible and well documented.
I like the buffer refactor, it makes the buffer more flexible and avoids the need for two pose buffers in in 🚀 launch, which is good.

@K-Meech K-Meech merged commit 5bebcdd into main May 14, 2026
1 check passed
@K-Meech K-Meech deleted the km/zero-gravity-data branch May 14, 2026 12:36
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