-
Notifications
You must be signed in to change notification settings - Fork 2
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
Reconceptualized biosample data collection #395
Conversation
Test Suite ReportTemplate Generation
Manifest Validation
|
717619c
to
480b6ed
Compare
2840d05
to
257a83a
Compare
And just as a note here that in Gray Foundation the structure is |
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.
We discussed this in our 1-1.
Key takeaways:
-this is great!
-let's consider pilot this manifest in the Sarin project and in Batch 5 of the NF1 biobank project (cc @jaybee84 )
-let's take a look at the 2023 Sundby paper for any updates that we could incorporate
-let's share with NTAP to see if we can align/adopt parts of their clinical data model
-let's get feedback from Sundby and Rhodes on the NF1 CDM slack
-consider changing name of manifest as described in PR text (prefer Cohort over HumanIndividual)
-this is good to merge as-is and then we can update after feedback
Initial work on #334. Cross-check with https://dash.harvard.edu/bitstream/handle/1/32725809/Neurology-2016-Hanemann-S40-8.pdf?sequence=1&isAllowed=y.
These templates require a lot of changes in workflows that want to adopt them. I think switching over wholesale for NF is going to be difficult to manage because it's very different from HTAN/Gray Foundation, so I think the original idea was to selectively offer/pilot this for just 1-2 projects that might actually benefit.
This is a prototype (not production use yet) that we merge but need to followup with additional work of setup + testing and understand where to refine things as needed:
If above goes well, we'll actually:
The preview links to review are mainly:
So if you think these aren't great names for the templates, I based them on ADKP (though for Gray Foundation, HumanIndividualTemplate ~ CohortCoreTemplate, which I think sounds better). We can discuss the best template naming as well.
Update:
@cconrad8 is primary reviewer.