Skip to content

Create AML -> myeloid referral list#75

Merged
niamhat merged 9 commits into
developmentfrom
myeloid
Feb 4, 2025
Merged

Create AML -> myeloid referral list#75
niamhat merged 9 commits into
developmentfrom
myeloid

Conversation

@PatrickWeller

@PatrickWeller PatrickWeller commented Jan 24, 2025

Copy link
Copy Markdown
Contributor

This Pull Request is linked to a PR in Somatic Enrichment Nextflow and should be code reviewed by the same person.
See: https://github.com/AWGL/somatic_enrichment_nextflow/pull/54


Code Changes

This pull request adds code that creates a sample list csv file of any DNA samples with an aml referral with the word myeloid.
Named: samples_aml_to_myeloid__.csv
E.g.

AB1234,myeloid
AB5678,myeloid

This file is then ready for the new_referral.sh script to be ran at the end of the somatic_enrichment_nextflow.nf script.
This has been tested with a run with multiple aml samples.

@PatrickWeller PatrickWeller changed the title Create AML referral list Create AML -> myeloid referral list Jan 24, 2025
@PatrickWeller PatrickWeller requested a review from niamhat January 28, 2025 08:50
@niamhat

niamhat commented Feb 4, 2025

Copy link
Copy Markdown
Contributor

Tests all okay, no changes to them made. Happy to merge. Works with: https://github.com/AWGL/somatic_enrichment_nextflow/pull/54

@niamhat niamhat left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All works. Tests work. Makes csv and runs end to end.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Makes samples csv as informed if aml referrals

@niamhat niamhat merged commit a4b0b8b into development Feb 4, 2025
@niamhat niamhat deleted the myeloid branch February 4, 2025 14:27
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.

6 participants