Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add record_provenance argument to sim_mutations; closes #2272 #2273

Merged
merged 2 commits into from
Jul 8, 2024

Conversation

petrelharp
Copy link
Contributor

I just followed what's done in ancestry.py here.

Copy link

codecov bot commented Mar 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.09%. Comparing base (fbd4bf2) to head (f5e1ae5).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2273   +/-   ##
=======================================
  Coverage   91.09%   91.09%           
=======================================
  Files          20       20           
  Lines       11918    11921    +3     
  Branches     2433     2435    +2     
=======================================
+ Hits        10857    10860    +3     
  Misses        581      581           
  Partials      480      480           
Flag Coverage Δ
C 91.09% <100.00%> (+<0.01%) ⬆️
python 98.70% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@jeromekelleher jeromekelleher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm. Should we update the changelog?

@benjeffery benjeffery force-pushed the omit_prov branch 3 times, most recently from 3f0e1ec to f351cf2 Compare July 8, 2024 14:32
@mergify mergify bot merged commit 42b99c0 into tskit-dev:main Jul 8, 2024
16 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants