Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update for individualID #430

Merged
merged 2 commits into from
Apr 19, 2024
Merged

Update for individualID #430

merged 2 commits into from
Apr 19, 2024

Conversation

anngvu
Copy link
Collaborator

@anngvu anngvu commented Apr 19, 2024

Related to FDS-1968.

So we should allow a list of IDs anyway, even aside from the above issue where I think schematic gets confused during submission when a field that's not supposed be a list ends up having commas (?).

@anngvu anngvu requested review from allaway and cconrad8 April 19, 2024 17:44
Copy link

PR Preview Action v1.4.7
🚀 Deployed preview to https://nf-osi.github.io/nf-metadata-dictionary/pr-preview/pr-430/
on branch gh-pages at 2024-04-19 17:45 UTC

Copy link

Test Suite Report

Template Generation

template result link
AnimalIndividualTemplate 😄 template link
BiospecimenTemplate 😄 template link
ClinicalAssayTemplate 😄 template link
EpigeneticsAssayTemplate 😄 template link
FlowCytometryTemplate 😄 template link
GenomicsAssayTemplate 😄 template link
GenomicsAssayTemplateExtended 😄 template link
HumanCohortTemplate 😄 template link
ImagingAssayTemplate 😄 template link
LightScatteringAssayTemplate 😄 template link
MethylationArrayTemplate 😄 template link
MRIAssayTemplate 😄 template link
PharmacokineticsAssayTemplate 😄 template link
PlateBasedReporterAssayTemplate 😄 template link
ProcessedAlignedReadsTemplate 😄 template link
ProcessedExpressionTemplate 😄 template link
ProcessedVariantCallsTemplate 😄 template link
ProteomicsAssayTemplate 😄 template link
ProtocolTemplate 😄 template link
RNASeqTemplate 😄 template link
ScRNASeqTemplate 😄 template link
UpdateMilestoneReport 😄 template link
WESTemplate 😄 template link
WGSTemplate 😄 template link

Manifest Validation

manifest result expectation
GenomicsAssayTemplate_0.csv 😄 Lists can be blank if attr not required using ‘list like’ rule
GenomicsAssayTemplate_1.csv 😄 Mixing blanks and regular list values works
GenomicsAssayTemplate_2.csv 😄 Conditional validation for attributes is currently not supported
ScRNASeqTemplate_0.csv 😄 Single list val works by using ‘list like’ rule
ScRNASeqTemplate_1.csv 😄 Fail because of missing data in required field libraryStrand

Copy link
Contributor

@allaway allaway left a comment

Choose a reason for hiding this comment

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

LGTM! One comment, we need to update the individualID attribute here to a stringList:
https://www.synapse.org/#!Synapse:syn16858331/tables/

Will we need to update all of the schemas for the project fileviews too?

@allaway allaway merged commit 5a53577 into main Apr 19, 2024
@allaway allaway deleted the patch/individualID branch April 19, 2024 18:48
@anngvu
Copy link
Collaborator Author

anngvu commented Apr 19, 2024

@allaway -- thanks, good reminder -- created ticket here; it does appear there needs to be a bit of cleanup for a couple of instances where ids were submitted as single long strings (e.g. "patient1, patient2, ...") before we can convert, because I'm getting warnings about table column sizes.

@cconrad8
Copy link
Contributor

Nice @anngvu ! I'm just curious where do these validation rules come from? are they specific to linkml or jsonld schema? wondering where to find "list like" and other validation rules

@allaway
Copy link
Contributor

allaway commented Apr 19, 2024

@anngvu , thanks for creating the ticket!

I am pretty sure this function will do what you need to handle the cleanup, though I'm not sure if it still works, since it has been a while!

@cconrad8
Copy link
Contributor

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants