Skip to content

[DRAFT] Stat recording improvements#3652

Draft
nycrat wants to merge 2 commits intomasterfrom
avah/stat_recording_improvements
Draft

[DRAFT] Stat recording improvements#3652
nycrat wants to merge 2 commits intomasterfrom
avah/stat_recording_improvements

Conversation

@nycrat
Copy link
Member

@nycrat nycrat commented Mar 16, 2026

Description

Implements the following improvements to stat recording:

  • --record_stats can trigger stat recording without needing other flags
  • run autoref/gamecontroller at ci_mode speed
  • change --ci_mode to take in duration to play instead of constant of 180s

Testing Done

todo

Resolved Issues

resolves #3633

Length Justification and Key Files to Review

Review Checklist

It is the reviewers responsibility to also make sure every item here has been covered

  • Function & Class comments: All function definitions (usually in the .h file) should have a javadoc style comment at the start of them. For examples, see the functions defined in thunderbots/software/geom. Similarly, all classes should have an associated Javadoc comment explaining the purpose of the class.
  • Remove all commented out code
  • Remove extra print statements: for example, those just used for testing
  • Resolve all TODO's: All TODO (or similar) statements should either be completed or associated with a github issue

@nycrat nycrat marked this pull request as draft March 16, 2026 05:05
@nycrat nycrat changed the title Stat recording improvements [DRAFT] Stat recording improvements Mar 16, 2026
@nycrat nycrat mentioned this pull request Mar 16, 2026
9 tasks
@Thunderbots Thunderbots force-pushed the avah/stat_recording_improvements branch 2 times, most recently from 27138b3 to c235d93 Compare March 16, 2026 17:30
@nycrat nycrat force-pushed the avah/stat_recording_improvements branch from 33c9ae4 to 4cd83fa Compare March 22, 2026 00:12
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.

Improve record stat

1 participant