Skip to content

Expand rocket launch data#84

Merged
K-Meech merged 27 commits into
mainfrom
km/rocket-launch-data
May 21, 2026
Merged

Expand rocket launch data#84
K-Meech merged 27 commits into
mainfrom
km/rocket-launch-data

Conversation

@K-Meech
Copy link
Copy Markdown
Collaborator

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

For #20

Expands rocket launch data, see extra clarifications from David at: #20 (comment) (also older comments at: #20 (comment) - most have been superseded, except that requiring 'Only measuring measuring their head speed with gaze on target')

Some extra notes:

  • Similar to the implementation for star collector, samples are taken for the save data every samplingIntervalSeconds. I've kept this parameter separate from the speedTime / gazeTime used for the main gameplay, so these can be adjusted separately.
  • As with other games, I've excluded any time periods including out of range time from the data. I've also incorporated this into the main gameplay that checks for speed / gaze steadiness - previously, it was possible to keep your gaze on the target and move your head to start the progress bar filling, then quickly walk out of range - as the buffer remained full of old data the progress bar could continue filling without the player being present. Now out of range time resets velocities to 0 and gaze steadiness to False.
  • I added a 'write sampled speeds' checkbox for easier debugging of saved speeds - should also be useful for Test robustness of save data #77
  • I've left the 'adaptive level' column out for now, and added a note to the description of: Rocket Launch: address playtesting feedback #81 . This will be simpler to add once the adaptive difficulty for rocket launch has been finalised.

@K-Meech K-Meech requested review from mmcleod89 and thompson318 May 19, 2026 09:56
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.

Looks good and works for me (with a small change for when using the mouse). Output csv looks correct and refactoring of speed and steadiness calculations is good.

A couple of minor comments re naming plus one thing that needs fixing when detecting player out of range when using the mouse instead of tracker.

Comment thread docs/rocket-launch.md Outdated
Comment thread projects/AstroBalance/Assets/Scripts/RocketLaunch/LaunchControl.cs Outdated
Comment thread projects/AstroBalance/Assets/Scripts/RocketLaunch/LaunchControl.cs
Comment thread projects/AstroBalance/Assets/Scripts/RocketLaunch/LaunchControl.cs Outdated
@K-Meech
Copy link
Copy Markdown
Collaborator Author

K-Meech commented May 20, 2026

Thanks for the comments @thompson318 ! - I think I've added all of those fixes now, so ready to review again 😄

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.

Looks good.

@K-Meech K-Meech merged commit ff6776c into main May 21, 2026
1 check passed
@K-Meech K-Meech deleted the km/rocket-launch-data branch May 21, 2026 07:35
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