-
Notifications
You must be signed in to change notification settings - Fork 113
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
NCBI Summary Report PR #1020
NCBI Summary Report PR #1020
Conversation
/gcbrun |
fill in missing sections in README.md
update Quantity dcid generation description
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.
There are critical errors in the data processing of this file. This is not ready for import as is. Please resolve critical errors being generated by the data commons json report tool. As part of resolving these errors the updated mcf files are uploaded to the shared drive. In addition, I noted where several the errors are being generated from - in detail - as comments on the test.sh and process.py files in this PR.
scripts/biomedical/NCBI_Assembly/Summary_Report/scripts/download.sh
Outdated
Show resolved
Hide resolved
scripts/biomedical/NCBI_Assembly/Summary_Report/scripts/process_test.py
Outdated
Show resolved
Hide resolved
scripts/biomedical/NCBI_Assembly/Summary_Report/scripts/process.py
Outdated
Show resolved
Hide resolved
scripts/biomedical/NCBI_Assembly/Summary_Report/scripts/test.sh
Outdated
Show resolved
Hide resolved
fix bugs in processing links to Taxon nodes
Add initiation of referenced Taxon nodes
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.
Critical errors remain in the formatting of the CSV file. The created tests in the process_test.py also do not run. Please fix
update subsections on new schema and caveats
update new schema subsection
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.
Looks good!
No description provided.