Skip to content

SURE - Discovering causal structure#385

Merged
jmafoster1 merged 57 commits into
CITCOM-project:mainfrom
Jake248Newman:main
Jul 10, 2026
Merged

SURE - Discovering causal structure#385
jmafoster1 merged 57 commits into
CITCOM-project:mainfrom
Jake248Newman:main

Conversation

@Jake248Newman

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread causal_testing/discovery/hill_climber.py Outdated
Comment thread causal_testing/__main__.py Outdated

@jmafoster1 jmafoster1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is looking good so far!

Comment thread causal_testing/main.py Outdated
@Jake248Newman Jake248Newman changed the title SURE - Discovering causal structures SURE - Discovering causal structure Jun 23, 2026
Jake248Newman and others added 3 commits June 23, 2026 10:43
Co-authored-by: Michael Foster <13611658+jmafoster1@users.noreply.github.com>
Comment thread tests/discovery_tests/test_hill_climber.py Outdated
@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.12500% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.98%. Comparing base (6dae63f) to head (67ca9ac).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
causal_testing/discovery/abstract_discovery.py 97.14% 3 Missing ⚠️
causal_testing/__main__.py 96.29% 2 Missing ⚠️
causal_testing/discovery/hill_climber_discovery.py 98.21% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #385      +/-   ##
==========================================
+ Coverage   97.89%   97.98%   +0.09%     
==========================================
  Files          27       31       +4     
  Lines        1568     1838     +270     
==========================================
+ Hits         1535     1801     +266     
- Misses         33       37       +4     
Files with missing lines Coverage Δ
causal_testing/discovery/nsga_discovery.py 100.00% <100.00%> (ø)
...esting/estimation/abstract_regression_estimator.py 96.77% <100.00%> (+0.10%) ⬆️
..._testing/estimation/linear_regression_estimator.py 100.00% <ø> (ø)
...esting/estimation/logistic_regression_estimator.py 100.00% <100.00%> (ø)
...ing/estimation/multinomial_regression_estimator.py 100.00% <100.00%> (ø)
causal_testing/main.py 100.00% <100.00%> (ø)
causal_testing/specification/causal_dag.py 98.83% <100.00%> (+0.58%) ⬆️
causal_testing/testing/causal_effect.py 100.00% <100.00%> (+1.44%) ⬆️
causal_testing/testing/metamorphic_relation.py 100.00% <100.00%> (ø)
causal_testing/discovery/hill_climber_discovery.py 98.21% <98.21%> (ø)
... and 2 more

Continue to review full report in Codecov by Harness.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jmafoster1 jmafoster1 marked this pull request as ready for review July 8, 2026 15:47

@jmafoster1 jmafoster1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for your work on this @Jake248Newman. This looks fine to me, so feel free to merge today, unless you can see anything obviously missing. We can make separate PRs for any minor changes we need as part of the evaluation

@jmafoster1 jmafoster1 merged commit e6a19c7 into CITCOM-project:main Jul 10, 2026
16 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