Closed williambrandler closed 2 years ago
Merging #477 (4846dc0) into master (f9eda3e) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #477 +/- ##
=======================================
Coverage 93.66% 93.66%
=======================================
Files 95 95
Lines 4875 4875
Branches 457 457
=======================================
Hits 4566 4566
Misses 309 309
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 7dc8a8e...4846dc0. Read the comment docs.
Signed-off-by: William Brandler william.brandler@databricks.com
What changes are proposed in this pull request?
Split the genotype simulation notebook up into three:
This avoids having to always download 1000G data, which is the bottleneck for testing the code. Functions are split out into their own notebook, eventually this notebook can be deleted when the simulation is done by a library such as sim1000G
Added a tip that phenotypes and covariates should be sorted in same order as genotypes
How is this patch tested?
(Details)