-
Notifications
You must be signed in to change notification settings - Fork 25
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
Juntian Wei BCO submission.md #176
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.
@jwei2019
I've reviewed your updated BCO and made comments on what further needs to be fixed.
Please make all changes in the BCO editor, download, and update this pull request file with the new JSON (copy/paste- same process as before). Once all changes have been made, assign this pull request back to me and I will review again. Until then, assigning this pull request back to you.
"orcid": "https://orcid.org/0000-0002-8824-4637", | ||
"affiliation": "George Washington University", | ||
"contribution": [ | ||
"createdBy" |
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.
Incorrect contribution type. Please modify. See #148
"contributors": [ | ||
{ | ||
"name": "", | ||
"orcid": "", | ||
"affiliation": "", | ||
"contribution": [ | ||
"createdBy" | ||
], | ||
"email": "jwei48@gwmail.gwu.edu" |
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.
Why is this empty? See #108
"usability_domain": [ | ||
"This pipeline is well suited to analyze the small non-coding RNAs obtained from smallRNA-Seq, which enables the future researchers to understand the biological role of sncRNAs better. The pipeline integrates different modules and sources together, and is used to identify different types of sncRNAs, as well as predicting its biological roles and mRNA targets. ", | ||
"Adapted from: Giurato, Giorgio, Maria Rosaria De Filippo, Antonio Rinaldi, Adnan Hashim, Giovanni Nassa, Maria Ravo, Francesca Rizzo, Roberta Tarallo, and Alessandro Weisz. “IMir: An Integrated Pipeline for High-Throughput Analysis of Small Non-Coding RNA Data Obtained by SmallRNA-Seq.” BMC Bioinformatics 14 (December 13, 2013): 362. https://doi.org/10.1186/1471-2105-14-362.", | ||
"For all the tools with no version numbers provided, the most up-to-date versions are used. " |
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.
Great job specifying this. 👍
"bco_spec_version": "1.3.0", | ||
"provenance_domain": { | ||
"name": "iMir: An integrated pipeline for high-throughput analysis of small non-coding RNA data obtained by smallRNA-Seq", | ||
"version": "1.0.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.
good job fixing version.
"date": "10_16_2019" | ||
} | ||
], | ||
"contributors": [ |
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.
Why are the publication authors not added as contributors? See #113
Juntian Wei's final draft of BCO.