Skip to content

Conversation

@hyanwong
Copy link
Member

@hyanwong hyanwong commented Sep 2, 2019

…_tree()

Remove old implementation

change docs

Tidy comments and variable names

@codecov
Copy link

codecov bot commented Sep 2, 2019

Codecov Report

Merging #185 into master will decrease coverage by 0.05%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #185      +/-   ##
==========================================
- Coverage   93.93%   93.87%   -0.06%     
==========================================
  Files          11       11              
  Lines        2886     2859      -27     
  Branches      507      498       -9     
==========================================
- Hits         2711     2684      -27     
  Misses        145      145              
  Partials       30       30
Impacted Files Coverage Δ
tsinfer/inference.py 98.98% <100%> (-0.05%) ⬇️

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 2008cb3...6592b5a. Read the comment docs.

…_tree()

Remove old implementation

change docs

Tidy comments and variable names
@hyanwong hyanwong force-pushed the map-mutations-for-noninference-sites branch from 887b6ee to 6592b5a Compare September 2, 2019 14:03
@hyanwong
Copy link
Member Author

hyanwong commented Sep 2, 2019

So this is now the version without any reference to missing data.

@jeromekelleher
Copy link
Member

Perfect, thanks. I'll merge once CI passes.

@hyanwong
Copy link
Member Author

hyanwong commented Sep 2, 2019

Thanks. Seems done now.

@jeromekelleher jeromekelleher merged commit db19f5c into tskit-dev:master Sep 2, 2019
@hyanwong hyanwong deleted the map-mutations-for-noninference-sites branch September 2, 2019 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants