morinlab / GAMBLR.results

A collection of functions to access results of the Genomic Analysis of Mature B-cell Lymphomas
MIT License
0 stars 0 forks source link

Rewrite of get_gambl_metadata and new helper functions #62

Closed rdmorin closed 3 months ago

rdmorin commented 4 months ago

The main changes are:

exome_priority_meta = get_gambl_metadata(capture_protocol_priority = "Exome")
2908 capture samples are missing a value for protocol. Assuming Exome.                                                                                 
55 biopsies are missing from the biopsy metadata. This should be fixed!                                                                                
affected cohorts:  DLBCL_Weng,DLBCL_Jain
27 biopsies with discrepancies in the pathology field. This should be fixed!
10 biopsies with discrepancies in the time_point field. This should be fixed!

> table(exome_priority_meta$protocol)

      Capture      chromium  ctDNA_Genome         Exome        genome        Genome         polyA         PolyA 
           34             3             5          3213            68          1854            11          1108 
Ribodepletion 
         1235 
not_exome_priority_meta = get_gambl_metadata(capture_protocol_priority = "Capture")
2908 capture samples are missing a value for protocol. Assuming Exome.                                                                                 
55 biopsies are missing from the biopsy metadata. This should be fixed!                                                                                
affected cohorts:  DLBCL_Weng,DLBCL_Jain
27 biopsies with discrepancies in the pathology field. This should be fixed!
10 biopsies with discrepancies in the time_point field. This should be fixed!

> table(not_exome_priority_meta$protocol)

      Capture      chromium  ctDNA_Genome         Exome        genome        Genome         polyA         PolyA 
           52             3             5          3195            68          1854            11          1108 
Ribodepletion 
         1235 
rdmorin commented 3 months ago

Would someone be able to test this again now that I've patched a few more bugs? It was a bit more involved to implement properly but it looks like the format (long/wide) is working as expected.