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

Fixes cycle basis for multiedges #27563

Closed
abhayptp opened this issue Mar 28, 2019 · 8 comments
Closed

Fixes cycle basis for multiedges #27563

abhayptp opened this issue Mar 28, 2019 · 8 comments

Comments

@abhayptp
Copy link

It fixes #27538. Now, cycle_basis() for multiedges works correctly.

sage: G = Graph([(1,2),(2,3),(2,3),(3,4),(1,4),(1,4),(4,5),(5,6),(4,6),(6,7)],multiedges=True)
sage: G.cycle_basis()
[[1, 4], [2, 3], [4, 3, 2, 1], [6, 5, 4]]

Currently, I am importing Graph from sage.graphs.graph in cycle_basis function to create instance of Graph. Suggestions are invited for any alternate approach.

CC: @dcoudert @rajat1433

Component: graph theory

Branch/Commit: u/gh-abhayptp/fixes_cycle_basis_for_multiedges @ b2c82ec

Reviewer: David Coudert

Issue created by migration from https://trac.sagemath.org/ticket/27563

@abhayptp abhayptp added this to the sage-8.8 milestone Mar 28, 2019
@abhayptp
Copy link
Author

@abhayptp
Copy link
Author

Commit: b2c82ec

@abhayptp

This comment has been minimized.

@abhayptp
Copy link
Author

New commits:

b2c82ecFixes cycle basis for multiedges

@abhayptp

This comment has been minimized.

@abhayptp
Copy link
Author

comment:4

Duplicate. See #27538.

@abhayptp abhayptp removed their assignment Mar 29, 2019
@dcoudert dcoudert removed this from the sage-8.8 milestone Mar 29, 2019
@dcoudert
Copy link
Contributor

Reviewer: David Coudert

@dcoudert
Copy link
Contributor

comment:8

We can close this ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants