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

Changed output format to mimic stable32's #100

Merged
merged 1 commit into from
Jul 26, 2019

Conversation

fmeynadier
Copy link
Contributor

Followup on #97

Not sure if you wish to include that in a rc2 or prefer to wait, this is up to you !

  • alpha evaluation not implemented yet, so "NaN" in this column for the moment.
  • uncertainty split in 2 to give min and max values
  • all stats share same output format (contrary to what stable32 does,
    eg. tierms vs adev).
  • introduced "digits" parameter to tune the number of significant digits, as suggested.
  • introduced "header_params" parameter to allow an arbitrary dict to be written in headers, as suggested.
  • "# " is added at the start of header lines. This is different from stable32 but I think it is wishable.

- alpha evaluation not implemented yet, so "NaN" in this column
- uncertainty split in 2 to give min and max values
- all stats share same output format (contrary to what stable32 does,
eg. tierms vs adev).
@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 84.095% when pulling 9347579 on fmeynadier:stable32_compat into f0a29b8 on aewallin:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 84.095% when pulling 9347579 on fmeynadier:stable32_compat into f0a29b8 on aewallin:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 84.095% when pulling 9347579 on fmeynadier:stable32_compat into f0a29b8 on aewallin:master.

@aewallin aewallin merged commit e025fa4 into aewallin:master Jul 26, 2019
@fmeynadier fmeynadier deleted the stable32_compat branch July 26, 2019 11: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.

3 participants