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

Totals Clarity in Web Results #629

Closed
blissend opened this issue Jul 21, 2017 · 3 comments
Closed

Totals Clarity in Web Results #629

blissend opened this issue Jul 21, 2017 · 3 comments

Comments

@blissend
Copy link

The front page of http://locust.io/ shows a web example of results where the totals row is a bit confusing. Some of the columns in the totals row is a sum of all numbers but not for others. For example the average column shows a total that is obviously not just a sum of all numbers. What is it a total of then? Perhaps it should be removed? This data isn't provided when you exit out of the process which gives text results without totals for that.

I found this on v0.7.5 and 0.8a2 available via pypi.

@cgoldberg
Copy link
Member

Perhaps it should be removed?

I agree.. I consider the presence of "Total" for every column to be a bug and am labeling this issue as such.

my $.02 on this in Slack was:

"Totals should probably just be removed from the columns that represent timings because they don't really make any sense in that context. Is a sum of averages useful to users? (I doubt it)"

@heyman
Copy link
Member

heyman commented Aug 18, 2017

I agree that the wording "Total" is bad. However I do think it's useful to see an average of all the average response times. For example, if one has a lot of endpoints it might be easier to spot if one endpoint suddenly becomes really slow, by looking at the total average, instead of scanning the whole list.

What if we were to change the label to "Aggregated"? Then at least it wouldn't be untrue, even though we use different aggregation methods.

@cyberw
Copy link
Collaborator

cyberw commented Oct 18, 2019

Closing this due to lack of activity.

@cyberw cyberw closed this as completed Oct 18, 2019
heyman added a commit that referenced this issue Oct 24, 2019
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

4 participants