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
Demes import #710
Demes import #710
Conversation
@@ -0,0 +1,24 @@ | |||
description: Two deme model with migration and size changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It'd be very cool if demes made its own examples available as package data.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree - should open an issue for that over in the demes repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are some examples perhaps, now that I think of it. I can see if there are and try to use some of those instead of having the example yaml file here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I'm just not sure they go in via "pip install".
This looks great. Just a few comments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is looking good. Just one more doc string thing, then we can probably squash and merge. We can update the manual in a later PR.
@@ -535,3 +525,16 @@ def _timed_events(self): | |||
self.set_migration_rates, | |||
]: | |||
yield i | |||
|
|||
|
|||
def from_demes( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's get the "versionadded" bit here, too, as this is the user-facing function.
Awesome--thanks! |
First steps to get demes into fwdpy11, updated from the previous
demes2fwdpy11
repo.