Codecov Report

Merging #947 into master will decrease coverage by 0.02%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##           master     #947      +/-   ##
==========================================
- Coverage   88.05%   88.03%   -0.03%     
==========================================
  Files          16       16              
  Lines        8457     8473      +16     
  Branches     1744     1748       +4     
==========================================
+ Hits         7447     7459      +12     
- Misses        504      506       +2     
- Partials      506      508       +2     

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ae6402c...b9976f4. Read the comment docs.

Sorry @ivan-krukov, this seems to have slipped under the radar somehow. Would it be possible to bring this up to date please? We have some benchmarks now, so it should be easy to quantify the improvements.

@ivan-krukov do you think we can bring this up to date and get ready for merging? The merge window for msprime 1.0 is quickly approaching, and we'd need to have this sorted out in the next week if it's going to get in for the release. It would be a shame for it to miss out, since it's a significant perf improvement.

Closing this as it's a long way behind the head of main. It would be nice to incorporate this idea back into the DTWF simulation at some point, as it should have a significant impact.