-
Notifications
You must be signed in to change notification settings - Fork 86
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
port to slim v4 #1326
port to slim v4 #1326
Conversation
fa6a17c
to
80c805e
Compare
Codecov Report
@@ Coverage Diff @@
## main #1326 +/- ##
=======================================
Coverage 99.67% 99.67%
=======================================
Files 111 111
Lines 3674 3674
Branches 473 473
=======================================
Hits 3662 3662
Misses 8 8
Partials 4 4
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
okay, well to review this properly i'm going to have to start by reading the new time units section! |
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.
looks like a simple enough conversion to v4, but there is definitely going to be breakage.
should average generation time be recorded in the metadata (as in the pyslim examples)? currently everything is WF so perhaps not?
"inds=p"+pop+".sampleIndividuals("+n+"); " + | ||
"sim.treeSeqRememberIndividuals(inds);}", | ||
g, g); | ||
if (n > 0) { |
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.
what happens if n <= 0?
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.
MesserLab/SLiM#345 happens
Where?
No, everything is WF, so one tick = one generation. (well, actually it equals |
okay sounds good @petrelharp! |
With these changes, tests now pass using the new version of SLiM on my computer at least. Mostly this just required changing "generation" to "tick"; there was only one thing that was hard to track down.