[EP-tech] Charts Visualization on IRStats - Abstract Pages - Chrome
David R Newman
drn at ecs.soton.ac.uk
Sat Jun 20 10:37:36 BST 2020
Hi Tomasz,
The eprints (rather than eprintsug) is the one used for generating new
releases of IRStats2 on the Bazaar (although it has been a while) but it
looks like the eprintsug has be updated by Rory more recently. I think
on balance I would submit a pull request to eprintsug.
We probably need to sort some consensus out on this, as having both
versions is unhelpful. I also have some local fixes that I have made to
IRStats2 that I have not had a chance to consider if they are worth
pushing back. Like everything, it is a case of finding the time to sort
these sorts of things out and then getting the best-placed people
together to organise things.
Regards
David Newman
On 19/06/2020 23:23, Tomasz Neugebauer via Eprints-tech wrote:
>
> I finally figured it out! That was not an easy one. The issue is
> with how the Google Graph libraries are initialized/loaded in IRStats2.
>
> I added explicit load and google.charts.setOnLoadCallback functions
> every time I need to generate charts, rather than assuming that the
> Google library is loaded before each new google.visulization call, and
> that solved the problem. Charts show up consistently and quickly, and
> I was able to switch to the more recent version of the Google charts
> library in the process. In fact, I had to, because only since version
> 45 are we allowed to call “load” more than once, which is be the case
> when we generate visualizations of different types on the same page in
> the ‘more statistics’ sections. I can prepare a pull request on
> github for this fix, but which version (I saw an IRStats2 on eprints
> and eprintsug) of the code on GitHub is the one that’s best/most
> recent to make the pull request on?
>
> Tomasz
>
> *From:*eprints-tech-bounces at ecs.soton.ac.uk
> <eprints-tech-bounces at ecs.soton.ac.uk> *On Behalf Of *Tomasz
> Neugebauer via Eprints-tech
> *Sent:* June 18, 2020 3:59 PM
> *To:* eprints-tech at ecs.soton.ac.uk
> *Subject:* Re: [EP-tech] Charts Visualization on IRStats - Abstract
> Pages - Chrome
>
> Sorry, I provided the wrong line number in the trace, same command,
> but the one on this line (263):
>
> https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Feprints%2Firstats2%2Fblob%2F74ce563ff855f7ba9702c4cd98532e95a17166a3%2Flib%2Fstatic%2Fjavascript%2Fauto%2F90_irstats2.js%23L263&data=01%7C01%7C%7C7efe6b326e5b42427fc608d814fd92ee%7C4a5378f929f44d3ebe89669d03ada9d8%7C0&sdata=HlCZnBABBVNviUIokYzH7WYHPFiW7RBh0%2FNylP%2F4UrQ%3D&reserved=0
> <https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Feprints%2Firstats2%2Fblob%2F74ce563ff855f7ba9702c4cd98532e95a17166a3%2Flib%2Fstatic%2Fjavascript%2Fauto%2F90_irstats2.js%23L263&data=01%7C01%7C%7C7efe6b326e5b42427fc608d814fd92ee%7C4a5378f929f44d3ebe89669d03ada9d8%7C0&sdata=HlCZnBABBVNviUIokYzH7WYHPFiW7RBh0%2FNylP%2F4UrQ%3D&reserved=0>
>
> the call to new google.visulization.DataTable()
> inEPJS_Stats_GoogleGraph=Class.create(EPJS_Stats,
>
> Tomasz
>
> *From:*Tomasz Neugebauer
> *Sent:* June 18, 2020 3:52 PM
> *To:* eprints-tech at ecs.soton.ac.uk <mailto:eprints-tech at ecs.soton.ac.uk>
> *Subject:* Charts Visualization on IRStats - Abstract Pages - Chrome
>
> I’ve been trying to track down a problem with loading of download
> summary table visualization on abstract pages and I can’t seem to
> figure it out.
>
> The problem only seems to appear in Google Chrome, on Firefox and Edge
> all is well.
>
> On Chrome, however, the graph visualizations don’t appear on the
> abstract pages, most of the time.
>
> Clicking on “Reload” will make it appear every now and again, so the
> issue is intermittent to some extent; but mostly the loader just keeps
> spinning and it doesn’t appear.
>
> I have tried to track down the issue, and as far as I see, the XHR
> call to getData works fine, and data is returned without problems, and
> passed to the correct functions.
>
> The problem appears right at this line:
>
> var data = new google.visualization.DataTable();
>
> So line 328 in this file:
> https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Feprints%2Firstats2%2Fblob%2F74ce563ff855f7ba9702c4cd98532e95a17166a3%2Flib%2Fstatic%2Fjavascript%2Fauto%2F90_irstats2.js%23L328&data=01%7C01%7C%7C7efe6b326e5b42427fc608d814fd92ee%7C4a5378f929f44d3ebe89669d03ada9d8%7C0&sdata=jXjsizO26f%2F3hl0cao9JeNBRC%2B5AYv9zjxJ6qAwGyo4%3D&reserved=0
> <https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Feprints%2Firstats2%2Fblob%2F74ce563ff855f7ba9702c4cd98532e95a17166a3%2Flib%2Fstatic%2Fjavascript%2Fauto%2F90_irstats2.js%23L328&data=01%7C01%7C%7C7efe6b326e5b42427fc608d814fd92ee%7C4a5378f929f44d3ebe89669d03ada9d8%7C0&sdata=jXjsizO26f%2F3hl0cao9JeNBRC%2B5AYv9zjxJ6qAwGyo4%3D&reserved=0>
>
>
> When I put a “Break” (in the debugger in the browser) on this line, I
> see that it is reached, and the retrieved data (about downloads) is
> there in the function, but the browser almost never comes back from
> this line and reaches no subsequent break points. Sometimes it does
> come back, reaches the next line, and then stops, and then sometimes,
> it does keep going to the end and then the graph does display.
>
> I tried disabling the CoreRecommender, as there are some warnings in
> Chrome about SameSite settings/cookies on that (not sure what that is
> all about either), in case that was the cause, but it didn’t change
> anything. I also tried changing how Google Charts / JSAPI is
> included/loaded, the version number of the library, but nothing seems
> to fix it.
>
> Any ideas? Do you display a chart of downloads on abstract pages, and
> is it working well in Chrome?
>
> Tomasz
>
> ________________________________________________
>
> Tomasz Neugebauer
> Digital Projects & Systems Development Librarian / Bibliothécaire des
> Projets Numériques & Développement de Systèmes
> Library / Bibliothèque
> Concordia University / Université Concordia
>
> Tel. / Tél. 514-848-2424 ext. / poste 7738
> Email / courriel: tomasz.neugebauer at concordia.ca
> <mailto:tomasz.neugebauer at concordia.ca>
>
> https://eur03.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.concordia.ca%2Ffaculty%2Ftomasz-neugebauer.html&data=01%7C01%7C%7C7efe6b326e5b42427fc608d814fd92ee%7C4a5378f929f44d3ebe89669d03ada9d8%7C0&sdata=x%2FGit6fD%2F6gJLATwjQzbFKhl2V8cHdrNBIiWBAd26N4%3D&reserved=0
> <https://eur03.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.concordia.ca%2Ffaculty%2Ftomasz-neugebauer.html&data=01%7C01%7C%7C7efe6b326e5b42427fc608d814fd92ee%7C4a5378f929f44d3ebe89669d03ada9d8%7C0&sdata=x%2FGit6fD%2F6gJLATwjQzbFKhl2V8cHdrNBIiWBAd26N4%3D&reserved=0>__
>
> Mailing address / adresse postale: 1455 De Maisonneuve Blvd.
> W., LB-540-03, Montreal, Quebec H3G 1M8
> Street address / adresse municipale: 1400 De Maisonneuve Blvd.
> W., LB-540-03, Montreal, Quebec H3G 1M8
>
> _library.concordia.ca_
>
>
> *** Options: http://mailman.ecs.soton.ac.uk/mailman/listinfo/eprints-tech
> *** Archive: https://eur03.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.eprints.org%2Ftech.php%2F&data=01%7C01%7C%7C7efe6b326e5b42427fc608d814fd92ee%7C4a5378f929f44d3ebe89669d03ada9d8%7C0&sdata=mQeRSbZwKJiX8IkS8NsJzarZm%2FBU10UOaASaE%2BRn3yA%3D&reserved=0
> *** EPrints community wiki: https://eur03.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwiki.eprints.org%2F&data=01%7C01%7C%7C7efe6b326e5b42427fc608d814fd92ee%7C4a5378f929f44d3ebe89669d03ada9d8%7C0&sdata=7oww8C61J%2Bj4pXjOV1Z3sdIDRR8KporlRq6JIKQuVmk%3D&reserved=0
--
This email has been checked for viruses by AVG.
https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.avg.com%2F&data=01%7C01%7C%7C7efe6b326e5b42427fc608d814fd92ee%7C4a5378f929f44d3ebe89669d03ada9d8%7C0&sdata=VRaJq%2FZlSroepEuCUGC1O41yXicezldQyLJDeF4nKE8%3D&reserved=0
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mailman.ecs.soton.ac.uk/pipermail/eprints-tech/attachments/20200620/b916cfae/attachment-0001.html
More information about the Eprints-tech
mailing list