Skip to content

[WIP, ENH] Add native implementation of GES #146

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.

Sign up for GitHub

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

Jump to bottom
Draft
adam2392 wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

[WIP, ENH] Add native implementation of GES #146

adam2392 wants to merge 10 commits into from

Conversation

@adam2392
Copy link
Collaborator

@adam2392 adam2392 commented Oct 30, 2023 •
edited
Loading

Closes: #29 and moves us towards #143

Changes proposed in this pull request:

  • generic GES algorithm implementation to allow future extensions more easily

TODO:

  1. Add unit-tests against other implementations
  2. Hash out the API
  3. Add a short example, similar to the PC algorithm example

Before submitting

@adam2392
Start adding GES
c30303a 
Signed-off-by: Adam Li <adam2392@gmail.com>
@adam2392
Adding wip ges
11d5c01 
Signed-off-by: Adam Li <adam2392@gmail.com>
@adam2392
Almost there depending on pywhy-graphs
043c2a6 
Signed-off-by: Adam Li <adam2392@gmail.com>
@adam2392
Merge branch 'main' into ges
9ee8d2f
@adam2392 adam2392 mentioned this pull request Dec 6, 2023
Open
adam2392 and others added 6 commits March 11, 2024 09:40
@adam2392
Almost there
423b39b 
Signed-off-by: Adam Li <adam2392@gmail.com>
@adam2392
Merging main
ce7c7db 
Signed-off-by: Adam Li <adam2392@gmail.com>
@adam2392
Updating unit tests
6d1afa6 
Signed-off-by: Adam Li <adam2392@gmail.com>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
2beeee3 
for more information, see https://pre-commit.ci
@adam2392
Merge branch 'main' into ges
2c4ad14
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
d3a70da 
for more information, see https://pre-commit.ci
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Implement GES using causal-learn

1 participant

@adam2392