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

Added general form of transitive closure #1257

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Procatv
Copy link
Contributor

@Procatv Procatv commented Jul 29, 2024

Resolves #704

Bulids on PR #707 (Not pushed yet) to implement the general form of the transitive closure of a graph. Implemented function by following structure from @IvanIsCoding:

  • Condense the graph into strongly connect components (which we already implement)
  • Apply the DAG algorithm and get the transitive closure for the intermediate graph
  • Map the answer from the intermediate graph to the original graph

On my testing branch containing the #707 PR as well, all tests pass.

@IvanIsCoding
Copy link
Collaborator

CI should be unblocked now that #1258 is merged

@IvanIsCoding IvanIsCoding added this to the 0.16.0 milestone Nov 16, 2024
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.

Transitive closure
2 participants