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

add specimenID to bts:ProcessedAlignedReadsTemplate #415

Merged
merged 2 commits into from
Apr 2, 2024
Merged

Conversation

allaway
Copy link
Contributor

@allaway allaway commented Apr 2, 2024

SpecimenID is useful here as it often contains info about different replicates, treatments, etc, in cell line sequencing

@allaway allaway requested a review from anngvu April 2, 2024 16:32
Copy link

github-actions bot commented Apr 2, 2024

PR Preview Action v1.4.7
🚀 Deployed preview to https://nf-osi.github.io/nf-metadata-dictionary/pr-preview/pr-415/
on branch gh-pages at 2024-04-02 16:33 UTC

Copy link

github-actions bot commented Apr 2, 2024

Test Suite Report

Template Generation

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

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

@allaway
Copy link
Contributor Author

allaway commented Apr 2, 2024

Hmm, not sure if I broke something or the tests are broken..

ImportError: cannot import name 'TypeAliasType' from 'typing_extensions' (/opt/hostedtoolcache/Python/3.10.12/x64/lib/python3.10/site-packages/typing_extensions.py)

Copy link
Collaborator

@anngvu anngvu left a comment

Choose a reason for hiding this comment

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

Thanks -- you can ignore the test results, there's suddenly some dependency issue on GH Actions

Copy link
Collaborator

@anngvu anngvu left a comment

Choose a reason for hiding this comment

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

Actually, I just realized there may be another fix needed, since this inherits from BiologicalAssayDataTemplate, which has specimenID -- let me make suggestion

Copy link
Collaborator

@anngvu anngvu left a comment

Choose a reason for hiding this comment

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

OK, from my comments I see why it wasn't at that level originally -- all good!

@anngvu anngvu merged commit ee890ed into main Apr 2, 2024
@anngvu anngvu deleted the allaway-patch-2 branch April 2, 2024 17:22
@anngvu
Copy link
Collaborator

anngvu commented Apr 3, 2024 via email

@allaway
Copy link
Contributor Author

allaway commented Apr 3, 2024

OK, from my comments I see why it wasn't at that level originally -- all good!

I went thru the exact same thought process! :) I am happy to move it up a level if that makes sense more generally. I can imagine other assays might benefit from specimenID info.

Thanks for the quick review!

@allaway allaway mentioned this pull request Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants