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

Remove deprecation warning for multigraph edges / refactor symmetrization #4588

Closed
rlratzel opened this issue Aug 5, 2024 · 1 comment · Fixed by #4649
Closed

Remove deprecation warning for multigraph edges / refactor symmetrization #4588

rlratzel opened this issue Aug 5, 2024 · 1 comment · Fixed by #4649
Assignees
Milestone

Comments

@rlratzel
Copy link
Contributor

rlratzel commented Aug 5, 2024

The removal of multi edges is no longer supported in symmetrize.py for both single and multi GPU workflows. Instead, multi edges can be removed at the graph creation through the flag drop_multi_edges.

@rlratzel
Copy link
Contributor Author

This has expanded into a design change that involves moving the symmetrization step into C/C++/PLC.

@rlratzel rlratzel changed the title Remove deprecation warning for multigraph edges Remove deprecation warning for multigraph edges / reactor symmetrization Sep 17, 2024
@rlratzel rlratzel changed the title Remove deprecation warning for multigraph edges / reactor symmetrization Remove deprecation warning for multigraph edges / refactor symmetrization Sep 17, 2024
raydouglass pushed a commit that referenced this issue Oct 3, 2024
This PR updates the python API to symmetrize the edge list through the
CAPI for PLC algorithms. This PR also deprecates the legacy python
function symmetrizing the edge list

closes #4588
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 a pull request may close this issue.

2 participants