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

Add __copy__ and __deepcopy__ to DiscreteDemography #773

Merged
merged 1 commit into from Jun 16, 2021

Conversation

molpopgen
Copy link
Owner

No description provided.

@molpopgen molpopgen linked an issue Jun 15, 2021 that may be closed by this pull request
@molpopgen molpopgen force-pushed the copy_deepcopy_DiscreteDemography branch 2 times, most recently from 15dab7e to b979dbe Compare June 16, 2021 22:15
state on the C++ side.

* Add copy constructors to several C++ types to
  facilitate cloning demographic model state

* Exceptions raised when building migration matrix rows now
  report source/destination demes.
@molpopgen molpopgen force-pushed the copy_deepcopy_DiscreteDemography branch from b979dbe to f405746 Compare June 16, 2021 23:30
@molpopgen molpopgen marked this pull request as draft June 16, 2021 23:46
@molpopgen molpopgen marked this pull request as ready for review June 16, 2021 23:46
@molpopgen molpopgen merged commit d3c247e into dev Jun 16, 2021
@molpopgen molpopgen deleted the copy_deepcopy_DiscreteDemography branch June 16, 2021 23:47
molpopgen added a commit that referenced this pull request Jul 5, 2021
The __deepcopy__ handles the internal state on the C++ side.

* Add copy constructors to several C++ types to
  facilitate cloning demographic model state

* Exceptions raised when building migration matrix rows now
  report source/destination demes.
molpopgen added a commit that referenced this pull request Jul 19, 2021
The new class is completely independent of DiscreteDemography.

Closes #782
Closes #783

This PR also affects the implementation of changes made in
PRs #773 and #774
molpopgen added a commit that referenced this pull request Jul 19, 2021
#791)

The new class is completely independent of DiscreteDemography.

Closes #782
Closes #783

This PR also affects the implementation of changes made in
PRs #773 and #774
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 this pull request may close these issues.

DiscreteDemography needs copy/deepcopy support
1 participant