search

alimanfoo / anjl

A neighbour-joining library for Python.
https://alimanfoo.github.io/anjl
MIT License
0 stars 0 forks source link

Code cruft - only update lower triangle of the distance matrix #49

Closed alimanfoo closed 3 days ago

alimanfoo commented 3 days ago

https://github.com/alimanfoo/anjl/blob/6c568b0f7656acfc4df857301c4873854d29b91d/anjl/_dynamic.py#L410-L414

Potentially could only access and update the lower triangle.

alimanfoo commented 3 days ago

Will leave this alone, better to resolve by moving to use a condensed distance matrix.