Fixed numerical error for large number of taxa and added comments

stamatak / standard-RAxML

www.exelixis-lab.org

325 stars 192 forks source link

Fixed numerical error for large number of taxa and added comments #34

Closed lutteropp closed 8 years ago

lutteropp commented 8 years ago

There was a numerical error in the computation of prefix sums for random quartet generation. It occurred when the number of taxa was larger than 1000. I fixed this issue, now it works for numberOfTaxa up to 100000.

I also added some comments to the random quartets code.

stamatak commented 8 years ago

Hi Sarah,

Your version seems to be a couple of commits behind mine, can you please merge in my changes and then re-issue the pull request?

Thanks :-)

Alexis