Improved precision of printed results#16
Open
rotemdan wants to merge 1 commit intocmpxchg16:masterfrom
rotemdan:improve_result_precision
Open
Improved precision of printed results#16rotemdan wants to merge 1 commit intocmpxchg16:masterfrom rotemdan:improve_result_precision
rotemdan wants to merge 1 commit intocmpxchg16:masterfrom
rotemdan:improve_result_precision
Conversation
Author
|
OK, I found a similar PR but it seems to have since been reverted in the master branch. The only difference between mine and that one is that it changed the number formatting alignment to 13 and had the rates as fractions. I left the length at 10 and kept the rates to display as integers (I don't really think there's a need for fractions since the rates are usually at least in the hundreds or more). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I improved the printed results to account for fractional second and millisecond timing. The rates are still displayed as integers (though now calculated with floating point accuracy) but the elapsed time is now a float with 2 decimal places. Example: