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

XiyingDing BCO draft submission #197

Closed
wants to merge 2 commits into from
Closed

XiyingDing BCO draft submission #197

wants to merge 2 commits into from

Conversation

xzd5071
Copy link
Contributor

@xzd5071 xzd5071 commented Oct 17, 2019

Ready to be reviewed.

Copy link
Collaborator

@jpat1546 jpat1546 left a comment

Choose a reason for hiding this comment

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

@xzd5071
I've reviewed your updated BCO and made comments on what further needs to be fixed- only 2 more items. Please make all the changes in the BCO editor, download, and update this pull request with the new JSON (copy/paste- same process as before). One 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 while you make necessary changes.

]
},
"usability_domain": [
"TAP is an RNAseq data analysis pipeline that is superior in speed and accuracy. It uses four steps to classify and reorganize reads from data and detect the structural and splicing variants. By analyzing open data from the published researches, TAP's result is not only consistent with researches, but also has new discoveries, proving that it is a powerful pipeline."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please provide "This BCO was adapted by [paper name] by [author list] [doi/PMID]. See #101

@jpat1546 jpat1546 assigned xzd5071 and unassigned kee007ney and jpat1546 Oct 17, 2019
@xzd5071 xzd5071 closed this Oct 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants