Closed bschilder closed 2 years ago
Could have been inherited, @NathanSkene have you seen NGT before?
Hey @Al-Murphy, mean to tell you, @NathanSkene and I discussed this in our last meeting and he said he doesn't recall coming across this column before. So we think it makes sense to remove it from the colmap.
Removed!
1. Bug description
Just came across a situation where "NGT" was a column in one of the PGC GWAS, which indicates " the number of sub-cohorts in which the variant is genotyped". So all the numbers are of course very small. However MSS maps "NGT" onto "N", making it seem like sample sizes are super tiny!
Have you seen cases where "NGT" actually means "N", or was this mapping inherited from elsewhere? If both are possible, then perhaps we could at least add a warning saying "NGT" is ambiguous, or maybe just removing it from the mapping file entirely.
Console output
Expected behaviour
Fix all problems in biology.
2. Reproducible example
Code
3. Session info