perfherder graphs should show the subtest summary when we have a highlighted revision

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
3 years ago
3 years ago

People

(Reporter: jmaher, Unassigned)

Tracking

Details

(Reporter)

Description

3 years ago
right now for the subtest summary to display, we need to actively click on a dot on the graph.  I would prefer that if the URL passed in highlightedrevision=<rev>, that we display the summary for that revision.

issues:
* what if the rev matches >1 series- which series do we display
* need to watch out for non summary graphs

if needed we could have a url param for displaySubTestSummary=true, or maybe the signature.
So, I understand the need for this. I would prefer to solve this by allowing the "disclosed" revision to be passed in as a separate url parameter, and then for the tooltip + summary to be automatically displayed. I don't think this should be very hard.
When I was implementing the revision highlighting I first thought of showing the subtest summary and opening up the tooltip but the problem was, as Joel mentioned in the case of more than one match for a revision, but I'm up for implementing wlach's suggestion.
(Reporter)

Comment 3

3 years ago
sounds like a plan.  The more flexible we can make this the better.  I imagine another round of what makes sense after we get more things working.  It opens the door to our creativity and points out what should be obvious :)
(Reporter)

Comment 4

3 years ago
with the subtest summary being shifted to use compare mode, do we still think this is valid?
Let's WONTFIX this. I think we do still want the ability to persist a "selected" revision, but that's covered by bug 1153956.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.