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

Ability to specify percentiles you need for response time chart #2311

Closed
A1BOCO opened this issue Mar 1, 2023 · 4 comments
Closed

Ability to specify percentiles you need for response time chart #2311

A1BOCO opened this issue Mar 1, 2023 · 4 comments

Comments

@A1BOCO
Copy link
Contributor

A1BOCO commented Mar 1, 2023

Is your feature request related to a problem? Please describe.

The software I'm currently testing by locust requires 98% < 1.0 sec and 99.5 % < 2.0 sec response time performance. The performance requirement will vary based on software or project etc.. It would be better that percentiles on performance char is configurable not only median and 95% percentile.
Benefit of having this function on chart page is, we can see necessary percentile info in response time chart dynamically every minutes.

Describe the solution you'd like

Ideal solution would be to add new command line argument like attachment. If the parameters are not specified, default value median and 95% are used

locust -f locust.py --percentile1 0.8 --percentile2 0.99

image

then displays like this

  1. Performance chart
    locust2

  2. report page
    image

Describe alternatives you've considered

applying to stats page is alternatives, but it's overall percentiles not time series

locust4

Additional context

I have updated locust for the ability on my local repo folked from orignal.

@cyberw
Copy link
Collaborator

cyberw commented Mar 1, 2023

Sounds reasonably useful. I’d prefer --percentiles 0.8,0.99 as the parameter style though. PR welcome.

@cyberw
Copy link
Collaborator

cyberw commented Mar 1, 2023

Note that for csv reports you can already overwrite locust.stats.PERCENTILES_TO_REPORT https://docs.locust.io/en/stable/configuration.html#customization-of-statistics-settings

@A1BOCO
Copy link
Contributor Author

A1BOCO commented Mar 1, 2023

Thank you for the reply. I'll take this style "--percentiles 0.8,0.99" and will send PR.

@cyberw
Copy link
Collaborator

cyberw commented Mar 1, 2023

👍 Or maybe this too fits better as an attribute on the stats module, now that I think about it… the choicr between what is command line settings and code values is kind of arbitrary, but since stats config is already in code..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants