Bug #12478
closed
[crunchstat-summary] bad time formatting for jobs > 24 hours
Description
The x-axis labeling for all graphs in the HTML output is wrong as the job passes the 24 hour mark:
20h50m0s 22h13m20s 23h36m40s 1d190000 1d295000 1d3100000
In addition to fixing the bug, it'd be nice to choose more intuitive intervals for the scale if not too much work. e.g. instead of 1h23m20s use 1h30m or just 1h
Files
Updated by Tom Morris over 7 years ago
- Target version changed from Arvados Future Sprints to 2017-11-08 Sprint
Updated by Tom Clegg over 7 years ago
- File example-10m.png example-10m.png added
12478-duration-fmt @ 805b4e7f801853e69514b11942bc2e5f79bc9dbd
Updated by Tom Morris over 7 years ago
Looks great. I don't suppose you could be talked into creating a PR for the upstream too?
Updated by Tom Clegg over 7 years ago
Tom Morris wrote:
Looks great. I don't suppose you could be talked into creating a PR for the upstream too?
It could certainly be generalized a bit and turned into timeOffsetTicker. Probably would want it to work in milliseconds like the rest of the JS world. Not sure whether what I've done for our special case is close enough to the generalized version to justify going down the PR road...
Updated by Anonymous over 7 years ago
- Status changed from In Progress to Resolved
- % Done changed from 0 to 100
Applied in changeset arvados|commit:c4ac98db757e738431dbcde54ac738fb554e16e8.