Skip to content

fix megatron samples#810

Merged
anoronh4 merged 8 commits intodevelopfrom
feature/megatron_list_update
Sep 4, 2020
Merged

fix megatron samples#810
anoronh4 merged 8 commits intodevelopfrom
feature/megatron_list_update

Conversation

@anoronh4
Copy link
Copy Markdown
Collaborator

@anoronh4 anoronh4 commented Jun 25, 2020

Fixed mapping files megatron list for samples (consistent with LIMS and IGO endpoint now)
s_C_000624_N001_d
s_C_000624_P002_d
s_C_001385_M001_d

based on corrections communicated by the PM team. Further information stated below:
s_C_000624_N001_d - originally megatron list had three pairs of fastq from two requests: 06208_D_23 and 07951_C_55 which were believed to come from the same sample. 07951_C_55 was later revised as s_C_000624_N002_d.
s_C_000624_P002_d - one pair of fastqs assigned to this sample originally was later found to belong to s_C_000624_P003_d
s_C_001385_M001_d - one pair of fastqs assigned to this sample originally was later found to belong to s_C_001385_M003_d

all of the re-assigned fastqs were assigned to samples outside of megatron.

@anoronh4 anoronh4 requested a review from gongyixiao June 25, 2020 20:54
@gongyixiao gongyixiao added the enhancement New feature or request label Jun 26, 2020
@gongyixiao gongyixiao added this to the 1.3.2 milestone Jun 26, 2020
@gongyixiao
Copy link
Copy Markdown
Collaborator

gongyixiao commented Jun 30, 2020

  • Check with PMs about sample s_C_006624_P001_d, 2 pairs in megatron, 1 pairs in IGO endpoints.
    Update: Megatron is correct. IGO need to fix this in LIMS. Check back later in here: 07951_L_1 possibly missing fq pair in lims LimsRest#131 (comment)
  • Change the sample name with _d1 or _d2 to match the LIMS change
  • Confirm the rule of not using the re-sequenced samples, and find out in megatron if there is any more samples with multiple request IDs.
    Update: Fixed
  • One tumor was paired differently between megatron and Voyager. Need to confirm which one is correct.
    Tumor MegatronNormal VoyagerNormal
    s_C_000624_P001_d s_C_000624_N001_d s_C_000624_N002_d
    Update: stick with Megatron Normal.

@gongyixiao gongyixiao removed this from the 1.3.2 milestone Jul 16, 2020
@anoronh4
Copy link
Copy Markdown
Collaborator Author

anoronh4 commented Sep 3, 2020

logging additional changes:

Old CMO ID | New CMO ID
-- | --
s_AML_GCT_2T2 | s_C_006629_T005_d
s_AML_GCT_2T1 | s_C_006629_P002_d
s_AML_GCT_2T3 | s_C_006629_M002_d

These a different mapping was provided by PM but was revised for accuracy between PM and @anoronh4

@anoronh4
Copy link
Copy Markdown
Collaborator Author

anoronh4 commented Sep 3, 2020

changed s_C_001668_N001_d to s_C_001668_N002_d because IGO changed the sample name on Aug 12

@anoronh4 anoronh4 merged commit 88b89e3 into develop Sep 4, 2020
@gongyixiao gongyixiao deleted the feature/megatron_list_update branch September 24, 2020 19:59
@gongyixiao gongyixiao mentioned this pull request Oct 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants