Dxchel/big cycle test - #606
Merged
Merged
Conversation
- Uses a graph with 2001000 nodes - Checks with Tree-like graph - Uses same structure with a last edge that creates a cycle
Contributor
Author
|
Not sure about why the automatic tests are failing but it seems it's a carried failure from other changes, when testing the project there seems to be some failing tests that don't relate to the changes, here's the summary: |
ZigRazor
approved these changes
Jul 14, 2026
Comment changed to rerun PR workflow
Contributor
Author
|
PR workflow seems stuck after days, commited a small change to a comment so the workflow can rerun |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adding a big cycle test with 2001000 nodes for true and false cases
Test runs for 33s on local PC
Added cstdint library on failing header
closes #60