-
Notifications
You must be signed in to change notification settings - Fork 83
PBTA Histologies: Fusion filtering base (3 of N) #865
Changes from 5 commits
bd3a94d
bea7b1d
cef95e9
858370f
129c3d3
74b33b5
a3fb859
a92cd90
c3dfcc9
c45b630
318c607
4ad06a0
bca3e06
c4b3101
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -19,6 +19,14 @@ params: | |
label: "results folder for pbta-fusion-putative-oncogenic.tsv files" | ||
value: results | ||
input: string | ||
base_run: | ||
label: "1/0 to run with base histology" | ||
value: 0 | ||
input: integer | ||
base_histology: | ||
label: "Base histology file" | ||
value: data/pbta-histologies-base.tsv | ||
input: file | ||
|
||
--- | ||
|
||
|
@@ -69,8 +77,14 @@ fusion_calls<-read_tsv(file.path(root_dir,params$dataPutativeFusion)) | |
outputfolder<-params$outputfolder | ||
|
||
#### get histology file | ||
if ( params$base_run ==0 ){ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same comment here! |
||
clinical<-read_tsv(file.path(root_dir, params$histology), guess_max = 10000) %>% | ||
dplyr::select(Kids_First_Biospecimen_ID, Kids_First_Participant_ID, broad_histology) | ||
} else { | ||
clinical<-read_tsv(file.path(root_dir, params$base_histology), guess_max = 10000) %>% | ||
dplyr::select(Kids_First_Biospecimen_ID, Kids_First_Participant_ID, broad_histology) | ||
} | ||
|
||
|
||
# add broad_histology to fusion | ||
fusion_calls<-fusion_calls %>% | ||
|
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,5 @@ | ||
FusionName broad_histology count | ||
KIAA1549--BRAF Low-grade astrocytic tumor 109 | ||
KIAA1549--BRAF Low-grade astrocytic tumor 114 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm assuming these results changes are expected, but just going to bring them to attention so we can check. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yup, these are expected since some samples were updated to Low-grade. More info in Jo Lynne's comment here #865 (comment) |
||
C11orf95--RELA Ependymal tumor 25 | ||
EWSR1--FLI1 Mesenchymal non-meningothelial tumor 7 | ||
KIAA1549--BRAF Neuronal and mixed neuronal-glial tumor 6 | ||
EWSR1--FLI1 Mesenchymal non-meningothelial tumor 5 | ||
REV3L--FYN Diffuse astrocytic and oligodendroglial tumor 5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like the only difference here is whether
params$base_histology
orparams$histology
is used, so I would reduce your if statement to specify ahistology_file
variable and then supply that to that code chunk rather than repeating the code chunk twice.