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

Maintain consistent units in CLI output #1191

Closed
mcandre opened this issue Oct 7, 2019 · 2 comments
Closed

Maintain consistent units in CLI output #1191

mcandre opened this issue Oct 7, 2019 · 2 comments
Labels
evaluation needed proposal needs to be validated or tested before fully implementing it in k6 ux

Comments

@mcandre
Copy link

mcandre commented Oct 7, 2019

Please display consistent units, e.g. deci-second precision, in order to reduce visual jitter during performance runs.

@na-- na-- added evaluation needed proposal needs to be validated or tested before fully implementing it in k6 ux labels Oct 7, 2019
@na--
Copy link
Member

na-- commented Oct 7, 2019

You can change the time unit with summaryTimeUnit / --summary-time-unit / K6_SUMMARY_TIME_UNIT option, for example k6 run --summary-time-unit=ms script.js. That should help somewhat, even though there are no deci-seconds (only s, ms and us for now). You also currently can't specify the decimal precision or the left padding of the numbers, so alignment may still not be perfect if you have high variability.

Improving the alignment of the end-of-test summary in general is tracked in #1024. Also, we're currently working on a way to export the end-of-test summary in a JSON format, so follow #1168 if you want to know when you'd be able to do it and format it however it works for you.

All of that said, I don't know if we should just always use milliseconds by default when displaying the metric values in the end-of-test summary... It will probably be a better UX to not mix µs and ms values at least, that's the most annoying part for me personally. But some metrics like iteration_duration and group_duration will likely have much larger values than the http_req_* metrics, sometimes in the tens or hundreds of seconds, which doesn't make sense to display as milliseconds... 😕 So, maybe we should just stick to having a mix of ms and s values, unless the user explicitly requests microseconds by --summary-time-unit=us?

@na--
Copy link
Member

na-- commented Jan 27, 2021

I'm closing this issue, now that we have handleSummary() in k6 v0.30.0 (#1768), this can be left entirely at the discretion of the user.

An in any case, as I mentioned in #1806, in k6 v0.31.0 we'll adopt the latest version of the JS-based summary generation code from jslib.k6.io, originally submitted in grafana/jslib.k6.io#20. So any change requests for the default text summary k6 outputs should be opened in https://github.com/loadimpact/jslib.k6.io/issues now.

@na-- na-- closed this as completed Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
evaluation needed proposal needs to be validated or tested before fully implementing it in k6 ux
Projects
None yet
Development

No branches or pull requests

2 participants