-
Notifications
You must be signed in to change notification settings - Fork 767
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
American Meteorological Society: No longer uses Atypon #2997
Comments
AMS has switched publishing platforms from Atypon to KGL PubFactory: https://www.pubfactory.com/our-clients/. However, other non-AMS journals on this platform don't have this specific issue, because they don't put the word "Abstract" into their Abstract, e.g. https://thejns.org/view/journals/j-neurosurg/138/3/article-p587.xml. I'll create a new translator for all PubFactory journals, since one doesn't seem to exist yet, and fix the small Abstract issue that's specific to AMS journals at the same time. EM already works really well, as you mention, so should be pretty easy. |
Do we actually need a PubFactory translator, or should we just have an AMS translator that fixes Abstract? (Or should we just add some code to EM or the translation architecture that strips "Abstract" at the beginning of an abstract?) (Or should we just email them?) We seem to get perfectly good data from that JNS article, and that's really the ideal — having sites offer great embedded metadata, including PDF links, without our having to maintain translators. Ideally we'd be regularly deleting translators, not adding them. |
I think the reason they're including the word "Abstract" as a heading in their Abstract is so that that heading renders the same in the Full Text for OA articles, e.g. https://journals.ametsoc.org/view/journals/amsm/59/1/amsmonographs-d-18-0005.1.xml?tab_body=fulltext-display, where the As to why create a PubFactory translator vs. a translator just for AMS that fixes Abstract, and leaving EM to handle the other PubFactory journals - the only reason I could see to create a PubFactory translator is that detectWeb in EM doesn't handle adding multiple items, e.g. https://journals.humankinetics.com/view/journals/pes/pes-overview.xml. What do you think? |
It definitely doesn't need to be there — these are plain-text
We could add a translator that supported just the |
Great, I sent an email to PubFactory to ask if they could change this in their platform, and I'll work separately on a translator that's just for |
It's been a week and I didn't hear back from PubFactory, so I went ahead and let this translator cover both single articles and However, I decided to include single articles in this translator so that |
https://forums.zotero.org/discussion/98261/translator-update-needed-for-american-meteorological-society-ams-journals
#945 (comment)
Can probably just use EM with some tweaks (e.g., stripping "Abstract" from the abstract).
The text was updated successfully, but these errors were encountered: