Comments (4)
Shouldn't the probability of match and probability of mismatch sum to 1? If that is the case, then I suspect there is a bug.
In this code snippet:
p_e = mu;
if (is_match) {
p_e = 1 - (num_alleles - 1) * mu;
}
Probability of mismatch = mu
Probability of match = 1 - (num_alleles - 1) * mu
Their sum is not 1 when num_alleles
> 2.
Probability of match should be (1 - mu)
, unless probability of mismatch is (num_alleles - 1) * mu
.
EDIT: Please disregard the above. I didn't realise it's the Rosen & Paten (2019) formulation, which rescales the mutation rate based on the number of alleles at the site. So, not a bug.
from tskit.
I agree it is a weird parameterisation, and doesn't belong in the core HMM. We should make this type of rescaling optional at the Python level.
from tskit.
I'm just looking at how computation of emission probability is done in test_haplotype_matching.py
. It seems like it is going with a slightly different interpretation of mu
(when there is no scaling based on the number of alleles)?
if is_match:
p_e = 1 - mu
else:
p_e = mu / (n_alleles - 1)
tskit/python/tests/test_haplotype_matching.py
Line 382 in 2dae133
from tskit.
Maybe mu
has been rescaled somewhere else? There may well be inconsistencies here, though
from tskit.
Related Issues (20)
- Add support for MRCA based IBD HOT 2
- Documentation for IBD segments
- write_vcf should use the reference sequence for REF, if present
- Number of variable sites (not just num_sites) HOT 1
- Add 'mid' property to Interval class
- Simpler, more efficient interface for pair coalescence rates HOT 6
- clarity on use of tskit.union() HOT 4
- Should 2-locus ratio statistics return NaN on 0/0? HOT 4
- Installing tskit on Windows HOT 7
- Use reusable workflows across repos? HOT 2
- Error pip install tskit on Windows HOT 2
- Add node_is_sample array HOT 2
- TreeSequence.f2 is not symmetric with multiallelic sites HOT 3
- Bug assertion in `ts.allele_frequency_spectrum` HOT 8
- Folded AFS between branch and site modes differs by factor of two HOT 7
- edges lost after merging two trees HOT 20
- Codecov upload issues HOT 1
- keep_intervals() giving _tskit.LibraryError: Can't squash, flush, simplify or link ancestors... HOT 7
- Update GitHub upload/download artefacts
- Add XTable.drop_metadata HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from tskit.