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

Fixes Bug 1191509 - New signature reports tab to show graphs based on date histogram aggregations #2940

Merged
merged 1 commit into from
Aug 13, 2015

Conversation

Tchanders
Copy link
Contributor

No description provided.

@Tchanders
Copy link
Contributor Author

@AdrianGaudebert A few things about these graphs:

I wasn't sure what default options were best, so I just went for Product for now.

I had to populate elastic search with some extra stuff here in order to get different lines to show on the graphs, so I added three extra fake signatures with c. 2, 10 and 20 products (this many products may be unrealistic, but there might be this many of some other field). Metrics Graphics only shows the first five lines by default:
screenshot from 2015-08-06 16 57 01
(You can see by the legend that there are actually c. 20 lines of data here.)

Adding classes for the extra lines does make them display, but it looks too crowded:
screenshot from 2015-08-06 16 59 34
(...in any case you still get the same problem - if you add twenty extra classes but there are thirty lines, only the first twenty will show up.)

@Tchanders
Copy link
Contributor Author

We could sort by which lines have the highest counts and just show the top 5 (but we'd have to choose between highest average and highest maximum count). Or we could just show the graph with 5 lines but let the user know that there are more lines, and let them select which they want to see?


// Set the placeholder.
this.$selectElement.select2({
'placeholder': 'Aggregate on...',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/Aggregate on/Graph of. Or something better.

@adngdb
Copy link
Contributor

adngdb commented Aug 7, 2015

Let's show only the top 4 lines, and use those with the maximum total number of results. To find out which are the top results, add a facet on the field, and use the results from that facet: they will be sorted by descending number.

For example: http://socorro/supersearch/?_histogram.date=version&_facets=version

Returns:

{
  "facets": {
    "histogram_date": ["..."] ,
    "version": [
      { "term": "WaterWolf", "count": 42 },
      { "term": "NightTrain", "count": 41 },
    ]
  }
}

On the graph, if there are more than 4 different lines, add a note saying something like "Showing top 4 results out of XX, not showing: a, b, c... ".

@Tchanders
Copy link
Contributor Author

New version with those changes.

@adngdb
Copy link
Contributor

adngdb commented Aug 10, 2015

This is looking very very good! I'm excited to finally see graphs in the signature report page!!! :)

I think the only comment I have is that when you hover a particular point, the MG tip tells you the number of crashes but does not tell you what it is for (for example, if I'm looking at a graph of count of crashes per version, when I hover a point I'd like it to tell me that these numbers are for version X or Y).

And now for the actual last comment: where are my unit tests? :)

@Tchanders
Copy link
Contributor Author

@AdrianGaudebert Here they are!

@adngdb
Copy link
Contributor

adngdb commented Aug 12, 2015

Looking excellent! However, you are conflicting with @peterbe work in #2924, and he is going to merge that today. Once that PR is merged, can you please rebase this one and change your unit test code so that it matches the new way of mocking SuperSearch introduce in #2924?

@Tchanders
Copy link
Contributor Author

@AdrianGaudebert Good to squash?

@adngdb
Copy link
Contributor

adngdb commented Aug 13, 2015

Yes! Excellent! I'll merge as soon as it's ready to go. :)

@Tchanders
Copy link
Contributor Author

@AdrianGaudebert Done!

@adngdb
Copy link
Contributor

adngdb commented Aug 13, 2015

Heh, I just noticed: when you have several graphs and you hover a point in one of the graphs, it changes the top-right legend for all graphs! Not sure if that's a bug in your code or in MG, but that's not a blocker anyway. I'm merging this anyway, can you file a bug to figure that issue out?

adngdb added a commit that referenced this pull request Aug 13, 2015
Fixes Bug 1191509 - New signature reports tab to show graphs based on date histogram aggregations
@adngdb adngdb merged commit fff108a into mozilla-services:master Aug 13, 2015
@Tchanders
Copy link
Contributor Author

Oops, yes I should be able to fix that!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants