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

Edge buffer is lazily initialized #626

Closed
molpopgen opened this issue Jan 13, 2021 · 0 comments · Fixed by #627
Closed

Edge buffer is lazily initialized #626

molpopgen opened this issue Jan 13, 2021 · 0 comments · Fixed by #627

Comments

@molpopgen
Copy link
Owner

When we start a sim, we are not allocating a list size equal to the number of currently-alive (founder) nodes. Things tend to work because the common use case is to evolve a lot so the necessary allocations happen. But, for cases like #624, when we re-enter the evolve code path, we can fail to have the correct number of lists allocated when we simplify every generation, including the first. The result from the Python side is IndexError, which ultimately originates from a call to fwdpp::nested_forward_lists::head.

@molpopgen molpopgen added this to the 0.12.0 milestone Jan 13, 2021
@molpopgen molpopgen linked a pull request Jan 13, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant