ah - use new GT encoding #6822
Merged
ah - use new GT encoding #6822
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ample metadata tsv
meganshand
reviewed
Sep 15, 2020
| if (columns == null) { | ||
| columns = line; | ||
| } else { | ||
| values = line; |
There was a problem hiding this comment.
Is there only one line in the metrics file? If so, can you put in a check for that?
| @@ -30,67 +36,39 @@ public String getColumnValue(VariantContext variant, ProbeInfo probeInfo, String | |||
|
|
|||
| // This where the validation step (required vs not) lives -- fail if there is missing data for a required field | |||
There was a problem hiding this comment.
Should this comment be deleted too?
meganshand
approved these changes
Sep 16, 2020
kcibul
approved these changes
Sep 16, 2020
|
|
||
|
|
||
| public static String getGTString(final VariantContext variant) { |
There was a problem hiding this comment.
I'm so surprised this code doesn't exist anywhere? the VCFWriter must be doing (largely) the same thing?
There was a problem hiding this comment.
yes, but it is very embedded in the actual writing of the output and in htsjdk which for some reason i think it's more difficult for us to modify to make it modular.
| gt = RawArrayTsvCreator.GT_encoding.AA; | ||
| } else if (allele.basesMatch(probeInfo.alleleB)) { | ||
| gt = RawArrayTsvCreator.GT_encoding.BB; | ||
| if (alleleIndexes.size() == 2) { |
There was a problem hiding this comment.
what if it's not two? should we throw an exception?
| } | ||
| return gt.getValue(); | ||
| return gt == RawArrayTsvCreator.value_to_drop ? "null" : gt.getValue(); |
There was a problem hiding this comment.
do you mean the string null or an actual null?
There was a problem hiding this comment.
yes, the actual string "null". the tsv needs some value and the bq import will convert it to an actual null in the db. added to the comment to explain
meganshand
pushed a commit
that referenced
this pull request
Oct 6, 2020
* change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv
kcibul
pushed a commit
that referenced
this pull request
Jan 29, 2021
* change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv
kcibul
pushed a commit
that referenced
this pull request
Jan 29, 2021
* change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv
kcibul
pushed a commit
that referenced
this pull request
Feb 1, 2021
* change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv
kcibul
pushed a commit
that referenced
this pull request
Feb 1, 2021
* change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv
Marianie-Alphonse
pushed a commit
that referenced
this pull request
Feb 16, 2021
* change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv
Marianie-Alphonse
pushed a commit
that referenced
this pull request
Feb 16, 2021
* change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv
kcibul
pushed a commit
that referenced
this pull request
Mar 9, 2021
* change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv
kcibul
pushed a commit
that referenced
this pull request
Mar 9, 2021
* change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv
mmorgantaylor
pushed a commit
that referenced
this pull request
Apr 6, 2021
* change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv
mmorgantaylor
pushed a commit
that referenced
this pull request
Apr 6, 2021
* change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv
mmorgantaylor
pushed a commit
that referenced
this pull request
Apr 6, 2021
* change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv
mmorgantaylor
pushed a commit
that referenced
this pull request
Apr 6, 2021
* change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv
This was referenced Mar 17, 2023
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
change GTs to single character, drop hom ref, add sample metrics to sample metadata tsv