Skip to content
New issue

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

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

Already on GitHub? Sign in 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants