Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#Centipede Refactor stats reporting in preparation for upcoming new stats #844

Closed
wants to merge 1 commit into from

Conversation

copybara-service[bot]
Copy link
Contributor

@copybara-service copybara-service bot commented Dec 8, 2023

#Centipede Refactor stats reporting in preparation for upcoming new stats

Also:

  • Stop reporting timestamps in stats comparisons between experiments (as well as in non-experiment runs with --v=1 or higher). Timestamps weren't reported historically, and their value is dubious in this context. They were added primarily for .csv file dumping, and are actively used there.
  • Slightly the logged stats formatting for better readability. Evident in the updated test.

@copybara-service copybara-service bot force-pushed the cl/588972760 branch 3 times, most recently from 1f0feb1 to 8071e48 Compare December 8, 2023 06:43
@copybara-service copybara-service bot changed the title #Centipede Refactor stats reporting to prepare for upcoming new ones #Centipede Refactor stats reporting in preparation for upcoming new stats Dec 8, 2023
@copybara-service copybara-service bot force-pushed the cl/588972760 branch 10 times, most recently from ade0efa to b16e110 Compare December 13, 2023 19:48
…tats

Also:

- Stop reporting timestamps in stats comparisons between experiments (as well as in non-experiment runs with --v=1 or higher). Timestamps weren't reported historically, and their value is dubious in this context. They were added primarily for .csv file dumping, and are actively used there.
- Slightly the logged stats formatting for better readability. Evident in the updated test.

PiperOrigin-RevId: 588972760
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.

1 participant