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

Add date to charts in web UI #2678

Closed
2 tasks done
kentmartin73 opened this issue Apr 17, 2024 · 5 comments
Closed
2 tasks done

Add date to charts in web UI #2678

kentmartin73 opened this issue Apr 17, 2024 · 5 comments

Comments

@kentmartin73
Copy link

Prerequisites

Description

The charts produced by the UI are great for an overview of what happened when, but when doing load and stability tests that run for days, it is very difficult to see which day what happened. Adding the date to the X axis in addition to the time and/or the mouseover information would make the charts a lot easier to read in these circumstances

@cyberw
Copy link
Collaborator

cyberw commented Apr 17, 2024

I think that sounds like a useful option, maybe even something that could be automatically enabled once the test runs for more than 24. But someone needs to build it :)

@alejandro-carmena
Copy link

I would also say that it could be quite useful being able to zoom in on the charts, also for having a better visibility in the tests that run for days.

@andrewbaldwin44
Copy link
Collaborator

I think trying to detect when the test runs for more than 24hr to display the date adds too much overhead, not to mention adding the date to the x axis makes it quiet difficult to read. I would suggest displaying keeping the display on the x axis and add the full timestamp to the tooltip (on mouseover)
image
On the frontend it would be straightforward, we would just need to modify the stats and history in the backed to return a UTC timestamp instead of just the current time. What do you think @cyberw?

Also for zooming this is supported in echarts so I will add it 👍

@cyberw
Copy link
Collaborator

cyberw commented May 25, 2024

Sounds good!

@JavierUhagon
Copy link
Contributor

#2731 was merged, this should be closed, right? @cyberw

@cyberw cyberw closed this as completed May 27, 2024
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

5 participants