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

Part 3 v6 release: Update download script and doc #34

Merged
merged 12 commits into from
Jul 2, 2021
Merged

Conversation

kgaonkar6
Copy link

@kgaonkar6 kgaonkar6 commented Jun 29, 2021

🚨 Only merge this PR once #43 #44 is reviewed and merged.

Purpose/implementation Section

What scientific question is your analysis addressing?

Data release v6

What GitHub issue does your pull request address?

d3b-center/ticket-tracker-OPC#57

Directions for reviewers. Tell potential reviewers what kind of feedback you are soliciting.

Please review documentation for release in doc folder

Which areas should receive a particularly close look?

Data Release Checklist

  • Is the table in doc/data-file-descriptions.md up to date?
  • Is doc/data-format.md up to date?
  • Is doc/release-notes.md up to date?
  • Is download-data.sh up to date?
  • Was download-data.sh tested and did it complete without error?

@kgaonkar6 kgaonkar6 changed the title Part2: v6 release Part 3 v6 release: Update download script and doc Jun 30, 2021
Copy link
Member

@jharenza jharenza left a comment

Choose a reason for hiding this comment

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

Thanks @kgaonkar6 - will you update the release notes in the download per those updates?

The download otherwise works, but I would like to do one check on the histologies file later when I get a chance.

doc/data-formats.md Outdated Show resolved Hide resolved
doc/release-notes.md Outdated Show resolved Hide resolved
kgaonkar6 and others added 2 commits July 1, 2021 10:44
Co-authored-by: Jo Lynne Rokita <jolynnerokita@d3b.center>
Co-authored-by: Jo Lynne Rokita <jolynnerokita@d3b.center>
doc/release-notes.md Outdated Show resolved Hide resolved
Co-authored-by: Jo Lynne Rokita <jolynnerokita@d3b.center>
doc/release-notes.md Outdated Show resolved Hide resolved
Copy link

@runjin326 runjin326 left a comment

Choose a reason for hiding this comment

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

looks good to me!

@jharenza
Copy link
Member

jharenza commented Jul 1, 2021

Updated the following file to match our histologies files plus updates coming in d3b-center/ticket-tracker-OPC#48:

c734c0ce531024df3f2e4eae6f1ba602  efo-mondo-map.tsv

@jharenza
Copy link
Member

jharenza commented Jul 1, 2021

@kgaonkar6 I updated the release notes as well, and the md5sums for both efo and release files, and download checks out. Histologies should be the last update.

@kgaonkar6
Copy link
Author

ok updated histologies.tsv , all checks passed for me.

Copy link
Member

@jharenza jharenza left a comment

Choose a reason for hiding this comment

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

this looks ready!

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