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

improve readability of results text #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vmmello
Copy link

@vmmello vmmello commented Oct 15, 2021

Improve the readability of results by vertically aligning numbers and units in the same
column. Also improves aligning of 3 digit numbers.

E.g., before patch:

                  test1   test2   test3   test4   test5   test6   test7   test8   test9   test10  Average 
google            20 ms   19 ms   23 ms   21 ms   21 ms   19 ms   20 ms   21 ms   22 ms   23 ms     20.90
quad9             309 ms  1000 ms 233 ms  1000 ms 259 ms  273 ms  1000 ms 306 ms  275 ms  378 ms    503.30
freenom           368 ms  479 ms  499 ms  363 ms  392 ms  396 ms  478 ms  377 ms  393 ms  456 ms    420.10

After patch:

                     test1    test2    test3    test4    test5    test6    test7    test8    test9    test10    Average 
google               20 ms    19 ms    23 ms    21 ms    21 ms    19 ms    20 ms    21 ms    22 ms    23 ms       20.90
quad9               309 ms  1000 ms   233 ms  1000 ms   259 ms   273 ms  1000 ms   306 ms   275 ms   378 ms      503.30
freenom             368 ms   479 ms   499 ms   363 ms   392 ms   396 ms   478 ms   377 ms   393 ms   456 ms      420.10

Improve the readability of results by aligning numbers and units in the same
column. Also improve aligning of 3 digit numbers.
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