popsim-consortium / stdpopsim

A library of standard population genetic models
GNU General Public License v3.0
126 stars 87 forks source link

Move sweep API and extended events from `stdpopsim.ext` to `stdpopsim` #1569

Closed nspope closed 4 months ago

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.83%. Comparing base (e52b33e) to head (5ae0d02).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1569 +/- ## ========================================== - Coverage 99.83% 99.83% -0.01% ========================================== Files 131 130 -1 Lines 4340 4339 -1 Branches 595 595 ========================================== - Hits 4333 4332 -1 Misses 3 3 Partials 4 4 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

nspope commented 4 months ago

Does the added doc blurb look OK here @petrelharp ? I added in the API for extended events, but maybe we don't actually want these as they're not mentioned anywhere else in the docs (e.g. we could just include selective_sweep in api.rst)

petrelharp commented 4 months ago

I'm going to merge, and will consider the docs later.