Get parent tags to display child tag content in graphs

[jywarren]

We've just created a tag aliasing system in #871, so that content tagged with spectrometer, which has a tag.parent of spectrometry, appears on the spectrometry (parent) tag page:

https://publiclab.org/tag/spectrometry

However, the graph shown on that page does not show spectrometer-tagged content, just the listing below does. Let's fix it in the graph, so the graph should include all data shown on the graph shown at https://publiclab.org/tag/spectrometer.

I believe the graph draws on data from these two methods: https://github.com/publiclab/plots2/blob/master/app/models/drupal_tag.rb#L143-L169

Yes - the call to tag.weekly_tallies is here: https://github.com/publiclab/plots2/blob/master/app/views/tag/show.html.erb#L20

[manish310794]

@jywarren What if the hierarchy goes like this spectometry <- spectometer <- sub-category? Will the spectrometer tag page show content with tags spectometer and sub-category as well?

[jywarren]

Hi! No, I think only one level... To do more could lead us to circular and
infinite queries, right?

Thanks!

On Oct 28, 2016 9:29 AM, "Manish Agarwal" notifications@github.com wrote:

@jywarren https://github.com/jywarren What if the hierarchy goes like
this spectometry <- spectometer <- sub-category? Will the spectrometer
tag page show content with tags spectometer and sub-category as well?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#918 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJ4FkrP2A5bM0FV2LY0ZU0vM3Unufks5q4iLRgaJpZM4KfLzM
.

[manish310794]

Yes, you are right. It might lead to circular queries since it not a strict category-sub-category system.

[jywarren]

Hi, were you interested in taking this issue on? I'd be happy to offer some more input if so.

[manish310794]

@jywarren Yes I am interested to take this up.

[jywarren]

Super! Tell me if you need anything and thanks!

On Oct 31, 2016 1:13 AM, "Manish Agarwal" notifications@github.com wrote:

@jywarren https://github.com/jywarren Yes I am interested to take this
up.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#918 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJzkLUzp1ZuKZ49N2lAlmch-Z3-PDks5q5Xj9gaJpZM4KfLzM
.

[jywarren]

Reworking this into a plan to do some auto-tagging! #6367