Skip to content

Conversation

@servidge
Copy link
Contributor

Some small adjustments in one pull request.

  • Changed output type to stdout for --version and --csv-header. Addition to "send --json and --csv results to stdout and logs to stderr send --json and --csv results to stdout and logs to stderr #39"
  • decimal places at jitter and ping, better values with --share, this limits/extended ping and jitter to .2 decimal places at --csv / --json and stdout. Also it prevents 1.0909090909090908 ping times within csv and json and extend those values in rest of the data. (the initial reason for the change)
  • formatting, tabs, spaces and blank lines

Changed output type to stdout for --version and --csv-header.
Addition to "send --json and --csv results to stdout and logs to stderr librespeed#39"
better values with --share
The browser version doesn't really seem to use decimal places at ping but this
limits/extended ping and jitter to .2 decimal places at --csv / --json and stdout.
this prevent 1.0909090909090908 ping times within csv and json and extend those values in rest of the data.
The ping value could be adjusted to .0 to match the web version but this way the values are more accurate. +1 for the cli.
Formatting adjusted
@servidge
Copy link
Contributor Author

servidge commented Feb 15, 2022

Some of my modifications within the README.md are also contained in #47 from the other side of the water. ;) The rest has no overlap.

@maddie maddie merged commit 954e973 into librespeed:master Jul 8, 2022
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.

2 participants