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

Fix weird legend for hourly tweet stats #9

Open
gedankenstuecke opened this issue Dec 15, 2017 · 6 comments
Open

Fix weird legend for hourly tweet stats #9

gedankenstuecke opened this issue Dec 15, 2017 · 6 comments

Comments

@gedankenstuecke
Copy link
Owner

image

I don't exactly understand where these things come from, but should be fixable somehow.

@aashaka
Copy link
Contributor

aashaka commented Mar 1, 2018

I am interested in fixing this issue. I have started working on it and will send a PR in a few days :)

@gedankenstuecke
Copy link
Owner Author

That’s great, thanks so much!

@aashaka
Copy link
Contributor

aashaka commented Mar 9, 2018

To see the source of the problem, I extracted the hourly_stats data which was being fed to d3 in templates/tweet_display/index.html and used the MetricsGraphics Interactive Demo to tinker with it.

Turns out, the data works just fine when the x_accessor is a date, however, multiline graph with non-date x-values seem to have an inherent problem in the MetricsGraphics library itself.

In case we wish to really solve this issue without waiting for it to be corrected in the library, I can add linked graphs, which will look something like this. I can send in a PR for this if @gedankenstuecke agrees with it in the design sense.

Or we can wait if someone comes up with another hack. When I looked into the MetricsGraphics library, I found the following functions to be important for this issue: mg_process_multiple_y_accessors and mg_format_data_for_mouseover.

@gedankenstuecke
Copy link
Owner Author

I think we can keep the somewhat weird display for now, but would you want to submit an issue with this to the MetricsGraphics GitHub repository? If not I can do it too 😄

@aashaka
Copy link
Contributor

aashaka commented Mar 10, 2018

Sure, I will submit an issue to that repository!

EDIT: Here is the filed issue - metricsgraphics/metrics-graphics#831

@gedankenstuecke
Copy link
Owner Author

Awesome, thanks!

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

No branches or pull requests

2 participants