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

Modern UI: Lack of information about time and user count in charts #2467

Closed
2 tasks done
zifter opened this issue Nov 15, 2023 · 4 comments · Fixed by #2468
Closed
2 tasks done

Modern UI: Lack of information about time and user count in charts #2467

zifter opened this issue Nov 15, 2023 · 4 comments · Fixed by #2468
Labels

Comments

@zifter
Copy link

zifter commented Nov 15, 2023

Prerequisites

Description

In Modern UI in charts there is no information about user count and time

Expected:
image

Result:
image

Command line

Locustfile contents

-

Python version

3.11

Locust version

2.18.3

Operating system

linux

@zifter zifter added the bug label Nov 15, 2023
@cyberw
Copy link
Collaborator

cyberw commented Nov 15, 2023

Thanks for the report! @andrewbaldwin44 , when you have the time...

@cyberw
Copy link
Collaborator

cyberw commented Nov 15, 2023

@zifter Looking at your screenshot, the RPS is visible. Did you mean User count and timestamp? Or am I missing something :)

@zifter
Copy link
Author

zifter commented Nov 15, 2023

@cyberw yes, time and User count. Miswrite in description, title is still correct

@andrewbaldwin44
Copy link
Collaborator

@zifter @cyberw In my opinion I find the repetition of the user information to be a bit redundant when you can easily see the information in the last chart (because the charts are connected, hovering over any value in one chart highlights the value at the same timestamp in the other charts). Let me know what your thoughts are on this,

I added the timestamp to all tooltips :)

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

Successfully merging a pull request may close this issue.

3 participants