-
Notifications
You must be signed in to change notification settings - Fork 52
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
String db update to latest versions #360
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some links are broken and latest release for Protein is 11, better to use the latest release and not an old draft.
"identifier": "stringdb", | ||
"@type": "DataCatalog", | ||
"@id": "https://string-db.org/", | ||
"http://purl.org/dc/terms/conformsTo": "https://bioschemas.org/specifications/DataCatalog/0.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This link https://bioschemas.org/specifications/DataCatalog/0.3 does not work, the link should be https://bioschemas.org/profiles/DataCatalog/0.3-RELEASE-2019_07_01/
{ | ||
"@type": "Dataset", | ||
"@id": "https://string-db.org/#string.v11", | ||
"http://purl.org/dc/terms/conformsTo": "https://bioschemas.org/specifications/Dataset/0.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"license": { | ||
"@type": "CreativeWork", | ||
"name": "Creative Commons Attribution 4.0 International", | ||
"url": "https://creativecommons.org/licenses/by/4.0/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should not we use this URL as the "@id" as well?
"@id": "https://string-db.org/network/9606.ENSP00000379258", | ||
"http://purl.org/dc/terms/conformsTo": "https://bioschemas.org/specifications/DataRecord/0.2-DRAFT", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
], | ||
"@type": "bio:Protein", | ||
"@id": "https://string-db.org/network/9606.ENSP00000379258", | ||
"http://purl.org/dc/terms/conformsTo": "https://bioschemas.org/specifications/Protein/0.9-DRAFT", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The currentProtein profile version is https://bioschemas.org/profiles/Protein/0.11-RELEASE/ should it not be better to update the example to the latest release rather than using an old DRAFT profile?
Pull request updates the string-db markup to comply with the latest profiles for Bioschemas.