Skip to content
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

Update publication info graph with publication date using PubMed Id #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ImranAsif48
Copy link

The reason for this Pull request is that to find the timeline of discourse. I updated the code that fetched the PubMed ID's publication date using the PubMed API (In PubMed Java class). After fetching the publication date, I put it in the publication information graph with pav:authoredOn predicate. I also updated the dcterms:created to pav:createdOn because pav:createdOn is the date of creating the digital artefact or resource representation.

discourse. I updated the code that fetched the PubMed ID's publication
date using the PubMed API (In PubMed Java class). After fetching the
publication date, I put it in the publication information graph with
pav:authoredOn predicate. I also updated the dcterms:created to
pav:createdOn because pav:createdOn is the date of creating the digital
artefact or resource representation.
@egonw egonw self-assigned this Mar 3, 2021
@egonw egonw self-requested a review March 3, 2021 22:05
Copy link
Member

@egonw egonw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please have a look at the exception and other comments.

Regarding the webservice calls, this suggests we really want the publication data as part of the RDF.

This file will overcome the lot of webservice calls. The name of the
file is pubmed_data.json which contains the article information such as
title, authors, and publication date.
Copy link
Author

@ImranAsif48 ImranAsif48 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I resolved the issue by creating the pubmed json data file and uploaded it here. The issue was to fetch the pubmed API that gave the 500 HTTP response error. The pubmed publication date will be fetched from the json data file.This file will overcome the lot of webservice calls. The name of the file is pubmed_data.json which contains the article information such as title, authors, and publication date.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants