Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upPub Med Labs update to PubMed.js #1800
Comments
This comment has been minimized.
This comment has been minimized.
akinhwan
commented
Dec 19, 2018
Attached is a document with screenshots outlining what the user's complaints were. |
This comment has been minimized.
This comment has been minimized.
Thanks for being in touch! The underlying eutils API is remaining unchanged, right? |
This comment has been minimized.
This comment has been minimized.
akinhwan
commented
Dec 19, 2018
@adam3smith yes the underlying eutils API will remain unchanged as far as I know. will let you know if I hear otherwise! |
akinhwan commentedDec 19, 2018
•
edited
Hello Zotero translators!
A Zotero user has indicated that they are not getting the metadata properly ingested from Pub Med Labs. Pub Med Labs is the experimental site that will eventually replace PubMed. I am one of the developers on the project. This is my first time using Zotero personally, and would appreciate any an all input from the Zotero community.
Below are some of the changes I am planning to PR. but thought I'd open an issue to have any discussion/questions here.
update target regex so that PubMedTranslator.js fires on PubMedLabs (hereafter PML) urls
current PubMed.js translator looks for meta tag with name="ncbi_uidlist". However on the abstract page it contains single article IDs, while on the docsum page - multiple IDs. We have the same data in a meta tag with a different name. I could either update the PML meta or make the translator support PML meta.
translator.js has function getSearchResults that extracts from search results page array of tuples (article id, article title), and it is tied closely to PubMed HTML, and needs updating to support PML (in graceful manner based on data- attributes rather than CSS class names/HTML layout)