Closed FreekManders closed 1 year ago
In the cases where njs
does generate a tree, trying to plot it leads to very high memory usage (11gb) causing RStudio to crash.
Thanks! These three functions now check that there are at least three observations. I added the same check in fastme.bal
and fastme.ols
.
Hi Emmanuel, thank you for the package. When I run
njs
,bionj
, orbionjs
on a distance matrix calculated on only two sequences it sometimes causes RStudio to abort. Other times it throws an uninformative error, while at the same time returning a tree.While it doesn't make sense to create a tree on such a distance matrix, it should not cause RStudio to crash. The nj function does give an error with a helpful message:
It might be useful to update the first three functions to throw a similar error as
nj
.Example:
Session info: