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.

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

Bugs in vertex ordering algorithms. #45

Open
samuelsonric opened this issue Jan 11, 2025 · 3 comments
Open

Bugs in vertex ordering algorithms. #45

samuelsonric opened this issue Jan 11, 2025 · 3 comments
Assignees
@samuelsonric

Comments

@samuelsonric
Copy link
Collaborator

samuelsonric commented Jan 11, 2025 •
edited
Loading

The tests for the FlowCutter algorithm pass on my computer, but error (sometimes) when run by GitHub actions. FlowCutter is returning an empty file. Those tests are currently commented out.
@samuelsonric samuelsonric self-assigned this Jan 11, 2025
@samuelsonric
Copy link
Collaborator Author

samuelsonric commented Jan 17, 2025 •
edited
Loading

The Spectral ordering fails on singleton graphs due to an upstream bug in Laplacians.jl. That test is currently commented out.

@samuelsonric samuelsonric changed the title FlowCutter tests failing. Bugs in vertex ordering algorithms. Jan 17, 2025
@jpfairbanks
Copy link
Member

I didn't know about that package, but should be easy to patch upstream since there is only one ordering for the singleton graph!

@samuelsonric
Copy link
Collaborator Author

samuelsonric commented Jan 19, 2025 •
edited
Loading

The tests for the FlowCutter algorithm pass on my computer, but error (sometimes) when run by GitHub actions. FlowCutter is returning an empty file. Those tests are currently commented out.

Removed this algorithm for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@samuelsonric samuelsonric

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jpfairbanks @samuelsonric