Skip to content
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

DOC: causal_site documentation #126

Merged
merged 1 commit into from
Feb 6, 2024
Merged

Conversation

daikitag
Copy link
Collaborator

@daikitag daikitag commented Dec 4, 2023

Documentation for causal_site input.

Copy link

codecov bot commented Dec 4, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b2f780e) 100.00% compared to head (6720aca) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #126   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         8           
  Lines          330       330           
  Branches        46        46           
=========================================
  Hits           330       330           

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

@daikitag daikitag force-pushed the causal-site-doc branch 2 times, most recently from 2c52c72 to dce41de Compare December 4, 2023 23:33
@daikitag
Copy link
Collaborator Author

@jeromekelleher This has the documentation for specifying the causal sites, and would it be possible for you to see it whenever you have some time?

@jeromekelleher
Copy link
Member

Looks good, I think you just need to resolve the conflicts and squash your commits.

@jeromekelleher
Copy link
Member

I think we can squash these three commits to one @daikitag - the first two are very small changes

Add option to simulate causal sites by indicating the causal site IDs.
@mergify mergify bot merged commit b372bdc into tskit-dev:main Feb 6, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants