elifesciences / enhanced-preprints-import

Enhanced Preprints import system
1 stars 0 forks source link

MSID: 88777 Version: 2 DOI: 10.48550/arXiv.2302.14157 #2458

Open nlisgo opened 9 months ago

nlisgo commented 9 months ago

MSID: 88777

Version: 2

Preprint DOI: https://doi.org/10.48550/arXiv.2302.14157

Step 1. Awaiting reviews

Editorial to post reviews via hypothesis

Useful links:

For trouble shooting (e.g. no Docmaps available):

Step 2. Preview reviewed preprint

Production QC content ahead of publication

Instructions:

Useful links:

Step 3: Awaiting search reindex

This step adds the reviewed preprint to the homepage: https://elifesciences.org

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Useful links:

Step 4: Published! PDF requested

Waiting for PDF to be generated

Useful links:

Step 5: Introduce PDF to data folder and git repo

Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/

Step 6: Done!

KJT08 commented 9 months ago

Hi @fred-atherden this one isn't showing on Sciety, but I understand there is an issue with preprints on arXiv showing.

fred-atherden commented 9 months ago

Thanks @KJT08, yes the Sciety issue is not a problem although there is a separate problem with the reviews.

@acollings we have the same issue with this one where the AR has been added to the non-versioned URL (https://arxiv.org/abs/2302.14157 instead of https://arxiv.org/abs/2302.14157v2).

Screenshot 2024-01-30 at 08 35 04

Please could you take a look?

The URLS are:

fred-atherden commented 9 months ago

Just to add it looks like the reviews for the v1 made it onto the v1 link:

Screenshot 2024-01-30 at 08 37 33
acollings commented 9 months ago

Thanks @fred-atherden I've moved the reviews etc across across

fred-atherden commented 8 months ago

Here's what we now have:

Image

Unfortunately I see the AR is on the v1. I do also notice though that the docmap does not have the info on the revision here. I've added the info for the v2 to the unmatched spreadsheet. Let's see how that changes things once the information has made its way to the docmap

fred-atherden commented 8 months ago

@de-code - I wonder if you'd be willing to assist with updating the comments via hypothesis API here? (I'm assuming authorization is required to patch; I don't have credentials if so)

All the comments for the v2 are attached to the non-versioned link: https://arxiv.org/abs/2302.14157.

de-code commented 8 months ago

@de-code - I wonder if you'd be willing to assist with updating the comments via hypothesis API here? (I'm assuming authorization is required to patch; I don't have credentials if so)

Sure, I will try to get the credentials / an API key

fred-atherden commented 8 months ago

Sorry @de-code - I assumed you had the credentials, as a result of this comment. But re-reading it now I see that you just tested with your own account.

Maybe @acollings can help with the credentials?

acollings commented 8 months ago

(I'm not sure I can!! but I have a way of logging in to hypothes.is from an eLife account: let me know if that's what you need!)

de-code commented 8 months ago

Hi @acollings it would be good to quickly pair on it. If you have a way of logging in, then you should be able to create an API key if there isn't one already. I am not familiar with Hypothes.is permission system, whether there can be multiple people having permissions to update an annotation. In which case you could also add my account if that gave me the permissions.

de-code commented 8 months ago

Andy shared the API key with me...

Hi @fred-atherden just to avoid any potential confusion, we would like to move the following Hypothes.is annotations:

From https://arxiv.org/abs/2302.14157 to https://arxiv.org/abs/2302.14157v2

Is that correct?

fred-atherden commented 8 months ago

Thanks @de-code Yes - thats right, those ones need shifting onto the v2 link.

de-code commented 8 months ago

Done:

DocMaps dashboard is showing the update:

image

I've create a Google Sheet to move Hypothes.is annotation. I don't want to share it too widely due to the embedded API key. Perhaps next week could share it with you. It would allow you to update the URIs yourself.

fred-atherden commented 8 months ago

Amazing - thanks Daniel.

Yeah if you could share the google sheet with Andy and I, that would be really helpful.

fred-atherden commented 8 months ago

This one is ready to check

JGilbert-eLife commented 8 months ago

Looks like the theorem sections have been interpreted as figures rather than text and equations:

Image

Original PDF; tagging should be correct in the meca package:

Image

Equations 4 and 7 too:

Image

Image

de-code commented 8 months ago

Amazing - thanks Daniel.

Yeah if you could share the google sheet with Andy and I, that would be really helpful.

I have shared the Google Sheet with you two. I also updated the script to store the API token in a user property instead which makes it a bit more secure. But you will have to set the API token in order to use it (I or Andy can share it with you separately).

fred-atherden commented 8 months ago

@JGilbert-eLife, I've resolved the above issue with a workaround