If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

need interlink between avg and series graphs (new front end)

VERIFIED FIXED in 0.6

Status

Webtools Graveyard
Graph Server
VERIFIED FIXED
9 years ago
a year ago

People

(Reporter: alice, Assigned: rdoherty)

Tracking

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

9 years ago
graphs.mozilla.org/g2/ displays overall data, but each point on a given graph represents a test set which is then averaged into a single value.  The test set itself can be viewed on graphs.mozilla.org/g2/graph.html#type=series.  

Currently, you cannot go from a point on an overall graph and view the granular data that makes up that point.  You also can't go from a granular view to the overall view.  We need to be able to interlink so that end users can go back and forth between digging into the data and seeing the birds eye view.
Target Milestone: --- → 0.6
(Assignee)

Updated

9 years ago
Assignee: nobody → rdoherty
(Assignee)

Comment 1

9 years ago
Created attachment 340227 [details] [diff] [review]
Right click functionality to view test data

This patch adds the functionality desired. You can right click on a data point in a continuous graph and select 'View test' or right click on a discrete data point and select 'View continuous graph'.
Attachment #340227 - Flags: review?(anodelman)
(Assignee)

Comment 2

9 years ago
Created attachment 340373 [details] [diff] [review]
New patch for interlink

Here's the patch with the missing file.
Attachment #340227 - Attachment is obsolete: true
Attachment #340373 - Flags: review?(anodelman)
Attachment #340227 - Flags: review?(anodelman)
(Reporter)

Comment 3

9 years ago
Comment on attachment 340373 [details] [diff] [review]
New patch for interlink

For the most part, I like this.  It only hits a snag if you are looking at a continuous graph with multiple lines and then request to view the discrete graph - at that point you end up viewing a single discrete graph and you can't return to the multi-lined continuous graph that you were originally viewing.

Same issue comes up in the other direction (looking at two discrete graphs, attempt to look at continuous graph, end up at single continuous line and then can't get back to multi-test data discrete graph).

Is there any easy way to handle the multi-data sets issue?  Otherwise I'd be willing to go forward with this simply because it already adds so much missing functionality.
Attachment #340373 - Flags: review?(anodelman) → review+
(Assignee)

Comment 4

9 years ago
(In reply to comment #3)
> Is there any easy way to handle the multi-data sets issue?  Otherwise I'd be
> willing to go forward with this simply because it already adds so much missing
> functionality.

It's possible. We could have 2 options in the context menu, one for viewing 1 graph's data and another for viewing all of them. Passing around what tests to search for will be a little tough due to the db architecture, but I think it's possible.
(Reporter)

Comment 5

9 years ago
At least for the discrete multi-data set graphs it is unclear to me how it chooses which continuous graph to link to - so we should at least clarify that.  This probably also comes up with the continuous graphs.

So, making it clearer which graph link you are interlinking against would be good.
(Assignee)

Comment 6

9 years ago
It picks whichever graph point is closest to your mouse cursor, which is the same as the crosshairs. I could add the test id or name to the context menu to show what test you'll be sent to.
(Reporter)

Comment 7

9 years ago
Right - adding the test name/machine name to the context manu would make it a little less mysterious what is happening.
(Assignee)

Comment 8

9 years ago
Created attachment 340609 [details] [diff] [review]
Patch with test name & machine in contextmenu [checked in]

Here's a patch with everything, including the machine name and test name in the context menu.

I'm leaving the option to right click and select 'view all tests' out, I'd rather get this in now and add that later.
Attachment #340373 - Attachment is obsolete: true
Attachment #340609 - Flags: review?(anodelman)
(Reporter)

Updated

9 years ago
Attachment #340609 - Flags: review?(anodelman) → review+
(Reporter)

Comment 9

9 years ago
Comment on attachment 340609 [details] [diff] [review]
Patch with test name & machine in contextmenu [checked in]

Looks like a good compromise to me.

Any chance we could colour code this somehow with the colour used for the graph line that the context menu is associated with...
(Assignee)

Comment 10

9 years ago
(In reply to comment #9)
> Any chance we could colour code this somehow with the colour used for the graph
> line that the context menu is associated with...

I think the name & crosshairs are enough for now. File a bug if we really need it :)

Committed in revision 128:eb76442811bd.
(Assignee)

Updated

9 years ago
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
(Assignee)

Updated

9 years ago
Attachment #340609 - Attachment description: Patch with test name & machine in contextmenu → Patch with test name & machine in contextmenu [checked in]
Verified FIXED:

1. Loaded http://graphs-stage.mozilla.org/, and added "Tp3 on Windows XP".
2. Right-clicked on a point and chose "View continuous tests for Tp3 on qm-pxp03-cycles-5".
3. That yielded http://graphs-stage.mozilla.org/#type=series&testname=tp_loadtime&date=1180994130&machine=qm-pxp03-cycles-5, which was the series data.

Nice fix :-)
Status: RESOLVED → VERIFIED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.