petrelharp / treestats_ms

1 stars 1 forks source link

Minor updates #2

Closed jeromekelleher closed 5 years ago

jeromekelleher commented 5 years ago

Here's some minor tweaks to the treestats paper @petrelharp. Nothing of any particular importance here, just some tidy ups and some comments.

petrelharp commented 5 years ago

notation

Agree.

The connection with the Sankoff algorithm is neat and we should probably think about this a bit more and maybe return to it in the discussion. Are there classical phylogenetics algorithms we can speed up by maintaining vectors efficiently across trees?

Good point! The "peeling" algorithm for likelihood computation could totally be done this way. (concretely, suppose we have at each of a sequence of positions along the genome a set of observed discrete "phenotypes", which might be nucleotide states (so I should probably say genotypes), and a Markov transition matrix; compute the probability of the phenotypes given the tree sequence.

Might as well keep a version of the algorithms here, right?

That seems right to me, yes.

jeromekelleher commented 5 years ago

OK, great. Shall we merge this much?

petrelharp commented 5 years ago

Yes, looks great. You can merge also, right?

jeromekelleher commented 5 years ago

Yeah, I can merge. Just wondering what the protocol for updates should be. Shall we just push to master ourselves or open PRs and let the other review and merge if happy?

petrelharp commented 5 years ago

Either way, but the PR method is nice to see what's changed, if there's enough changes that it's worthwhile.

jeromekelleher commented 5 years ago

OK, let's assume so that if a PR is made it's up to the other person to merge it.