-
Notifications
You must be signed in to change notification settings - Fork 3
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
Single article line chart display (test.parascope.io) #113
Comments
Single article bubble chart display? You mean the line chart? XML wasn't initially included because the API call needs to include
which makes things a bit slower. Given that ALM is plenty fast again, I can include that, sure. The rest of the feedback sounds good too, thanks! |
You can also get the xml downloads without
See https://github.com/articlemetrics/lagotto/blob/master/app/models/sources/counter.rb#L19-L22 for details. |
We're talking about time-series data, right? Not just single point in time. Where in the API response would that be:
I only have |
You are right: https://github.com/articlemetrics/lagotto/blob/master/app/models/history.rb#L137-L150 For counter we have only @jenniferlin15 do you really want xml downloads? We would need to change the ALM API for that, which I would rather postpone until we have done the changes in the database backend. |
To be clear, the previous visualization had XML downloads, but it did that by making two requests to the API and combining the results: https://github.com/articlemetrics/alm-report/blob/2e378a5c1bf3eb1d35d4139fbb486aa96ebbf0c1/lib/alm_request.rb#L197-L261 |
Yes, my point is mostly about making this more efficient. Which might not be so important for a single article, where the API is probably fast enough. Another consideration is whether we want to include other sources in this, something we can do since mid-2014 since we collect the numbers. We could also display daily stats the first 30 days after publication in a separate chart. That is up to @jenniferlin15, and of course the amount of work that would be needed. |
Actually, sure, this is the single article visualization, which already requests the |
Perfect. |
I managed to address some of this feedback:
Leaving this one open, but maybe this is good for now? |
I think it's fine for now. Please just correct the capitalizatino of the
Thanks! On Tue, Dec 16, 2014 at 6:36 AM, Jure Triglav [email protected]
"The blessed will not care what angle they are regarded from,
|
The XML downloads have disappeared from the single article line chart display.
Please add this third line to the chart.
Also, the display needs to be cleaned up.
Title: Article Usage over Time
Axis Labels:
These should be located on the outside of the graph. The font also needs to be increased - currently not legible due to size.
Graph Legend:
Make the bubbles in the legend bigger and align the corresponding text with the items (currently too high).
Bubble hover over:
The fonts need to be increased - currently not legible due to size.
The text was updated successfully, but these errors were encountered: