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

Error when estimating linked exchangeabilities (GTRpmix) #261

Closed
mjamy opened this issue Jul 4, 2024 · 4 comments · Fixed by #284
Closed

Error when estimating linked exchangeabilities (GTRpmix) #261

mjamy opened this issue Jul 4, 2024 · 4 comments · Fixed by #284

Comments

@mjamy
Copy link

mjamy commented Jul 4, 2024

Dear IQ-TREE team,

I'm using IQ-TREE v2.3.5 to estimate linked exchangeabilities under the C60 profile mixture model following this tutorial. My alignment has 143 taxa and 7,855 sites.

However, the run crashed almost immediately with the following error:

ERROR: modelprotein.cpp:998: virtual void ModelProtein::init(const char*, std::string, StateFreqType, std::string): Assertion `nxs_model' failed.

I've attached the log file and input files.

Thanks!
Mahwash

iqtree-error.zip

@thomaskf
Copy link
Collaborator

thomaskf commented Jul 4, 2024

Hi Mahwash,

Thanks for your interest in using the new feature.

It should be "CPREV" instead of "cpREV". That means your command should be:

iqtree2 -s concat_143t_32g_prequal_ginsi_bmge.fasta -m GTR20+C60+G4 --link-exchange -te concat_143t_32g_prequal_ginsi_bmge_cpREV-C60-G.treefile -me 0.99 --init-exchange CPREV -nt 20

Thomas

@thomaskf
Copy link
Collaborator

thomaskf commented Jul 4, 2024

Thanks for reporting the issue. We will update the codes so that in the next version, the model name with the small letter will be accepted for the option "--init-exchange". For this version, please use the capital letter for the model name as the option "-init-exchange" input.

@roblanf
Copy link
Collaborator

roblanf commented Jul 4, 2024

@thomaskf to clarify further, for the current version should all init models be in all caps?

E..g if one wanted to use Q.plant as an initial model in the current version, should it be adjusted to Q.PLANT?

@thomaskf
Copy link
Collaborator

thomaskf commented Jul 4, 2024

@roblanf Yes, for the current version, the init model for the option "--init-exchange" needs to be in all caps (e.g. Q.PLANT instead of Q.plant).

thomaskf added a commit that referenced this issue Jul 5, 2024
@thomaskf thomaskf closed this as completed Jul 5, 2024
@thomaskf thomaskf mentioned this issue Jul 26, 2024
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

Successfully merging a pull request may close this issue.

3 participants