Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Assertion limit failed utree_distances #130

Closed
padpadpadpad opened this issue Nov 5, 2021 · 3 comments
Closed

Assertion limit failed utree_distances #130

padpadpadpad opened this issue Nov 5, 2021 · 3 comments

Comments

@padpadpadpad
Copy link

Hi all

I am using raxml-ng that I installed via conda (RAxML-NG v. 1.0.3) on a ~250bp alignment of 3000 OTUs.

It got quite far and then I got this error when i checked on it this morning.

raxml-ng: /opt/conda/conda-bld/raxml-ng_1626880669841/work/libs/pll-modules/src/tree/utree_distances.c:790: _cmp_splits: Assertion `limit' failed. Aborted (core dumped)

This was after running the command:

raxml-ng --msa alignment_ps_otu_99percent_filtered.fasta --model GTR+G --seed 2 --prefix raxml/myxo1 --threads 4 --outgroup TCGCTGGCCGAATCGGAAGGTCTGACTCCGCAGGAACTGATCAACGCAAAGCCGGTGGCTGCCGCGATCAAGGAATTCTTCGGATCCTCGCAGCTTTCGCAGTTCATGGACCAGAACAATCCGCTTTCGGAAGTCACCCACAAGCGCCGCGTTTCGGCACTTGGACCAGGTGGCCTGACCCGCGAGCGTGCTGGCTTCGAAGTACGC

Attached is my alignment and my log file. Any ideas much appreciated.
log_and_alignment.zip

@amkozlov
Copy link
Owner

amkozlov commented Nov 8, 2021

Hi @padpadpadpad,

thanks for reporting! There is a bug when trying to root a collapsed multifurcating tree. So until I fix it, the easiest workaround would to be resume the analysis without --outgroup option, and then run a separate raxml-ng --evaluate --tree myxo1.raxml.bestTree --outgroup X,Y if needed.

Side note: apparently, you are trying to infer a "tree of reads", which is generally a bad idea due to insufficient signal (see eg. this discussion). Please consider using phylogenetic placement instead: https://github.com/Pbdas/epa-ng

Side note 2: please don't use sequences as taxon names, this was really inconvenient to work with.

@padpadpadpad
Copy link
Author

Hi @amkozlov I shall look into everything you have suggested!

Thank you so much for your quick and in-depth response.

In response to side note 2, I am trying to create the phylogenetic tree from the output of a dada2 run which automatically names the sequences the same as the sequence itself. I will make sure to change this in future for tree making and reporting purposes. Sorry for the inconvenience!

amkozlov added a commit that referenced this issue Nov 19, 2021
…traint

- fix assertion when rooting a multifurcating tree (#130)
@amkozlov
Copy link
Owner

amkozlov commented Dec 7, 2021

fixed in v1.1.0

@amkozlov amkozlov closed this as completed Dec 7, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants