Bug #12478
closed
[crunchstat-summary] bad time formatting for jobs > 24 hours
Added by Tom Morris about 7 years ago.
Updated about 7 years ago.
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
- Target version changed from Arvados Future Sprints to 2017-11-08 Sprint
- Assigned To set to Tom Clegg
- Status changed from New to In Progress
Looks great. I don't suppose you could be talked into creating a PR for the upstream too?
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...
- Status changed from In Progress to Resolved
- % Done changed from 0 to 100
Applied in changeset arvados|commit:c4ac98db757e738431dbcde54ac738fb554e16e8.
Also available in: Atom
PDF