Bug with Arc Removal Move
Closed this issue · 2 comments
lutteropp commented
I discovered that ArcRemoval moves destroy the property that clv indices are consecutive... I tried fixing this by remapping network clv indices, but somehow this leads to problems with incremental likelihood computation.
It appears to work fine when I disable incremental likelihood computation, which makes me suspect it has something to do with the pmatrices. The pmatrix entries do only depend on the branch length and the likelihood model parameters, right?
stamatak commented
yes that is correct
…On 12.01.21 02:47, Sarah Lutteropp wrote:
I discovered that ArcRemoval moves destroy the property that clv indices
are consecutive... I tried fixing this by remapping network clv indices,
but somehow this leads to problems with incremental likelihood computation.
It appears to work fine when I disable incremental likelihood
computation, which makes me suspect it has something to do with the
pmatrices. The pmatrix entries do only depend on the branch length and
the likelihood model parameters, right?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#38>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGXB6RGZJ6ERNND57BNJBDSZOL2TANCNFSM4V6NXZDQ>.
--
Alexandros (Alexis) Stamatakis
Research Group Leader, Heidelberg Institute for Theoretical Studies
Full Professor, Dept. of Informatics, Karlsruhe Institute of Technology
www.exelixis-lab.org