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

evolvets argument compatibility #648

Closed
3 tasks done
molpopgen opened this issue Feb 7, 2021 · 0 comments
Closed
3 tasks done

evolvets argument compatibility #648

molpopgen opened this issue Feb 7, 2021 · 0 comments
Labels
tree sequences UI User interface
Milestone

Comments

@molpopgen
Copy link
Owner

molpopgen commented Feb 7, 2021

After merging #647, we need to make sure that various arguments are properly set. This issue tracks this. We likely need more tests and runtime exceptions.

Resetters and suppress_table_indexing. If a resetter is used, this argument must be False. This has been shown to be not necessary. Can revisit later if it bites us. ;)

  • Tracking mutation counts, neutral mutations, and the simplification interval. If 1 and 2, then 3 must be 1.
  • Ensure that table indexes are empty (during the simulation) if supress_table_indexing is True.
  • Ensure that table indexes are empty for any generations where simplification does not happen, even if suppress_table_indexing is False.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tree sequences UI User interface
Projects
None yet
Development

No branches or pull requests

1 participant