Skip to content

[pull] master from williamfiset:master#129

Merged
pull[bot] merged 4 commits intoUncodedtech:masterfrom
williamfiset:master
Mar 16, 2026
Merged

[pull] master from williamfiset:master#129
pull[bot] merged 4 commits intoUncodedtech:masterfrom
williamfiset:master

Conversation

@pull
Copy link

@pull pull bot commented Mar 16, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

williamfiset and others added 4 commits March 15, 2026 21:57
…st (#1294)

The adjacency matrix variant is redundant with BellmanFordEdgeList and
BellmanFordAdjacencyList. Tests that used it as an oracle now use
BellmanFordEdgeList instead.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
…1295)

Redundant adjacency matrix variants — the adjacency list versions
(LazyPrimsAdjacencyList, EagerPrimsAdjacencyList, TopologicalSortAdjacencyList)
are more efficient and already cover the same algorithms.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* Refactor BFS, remove FastQueue and Recursive duplicates

Remove BreadthFirstSearchAdjacencyListIterativeFastQueue (custom IntQueue
variant redundant with ArrayDeque) and BreadthFirstSearchRecursive (recursive
BFS with queue-passing hack). Refactor the remaining BFS implementation
with improved docs, final fields, and cleaner code style.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Rename BreadthFirstSearchAdjacencyListIterative to BreadthFirstSearchAdjacencyList

Drop "Iterative" from the class and file name now that the recursive and
fast-queue variants have been removed. Replace the 13-node example with
a simpler 8-node graph including a visual diagram.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Remove cost from Edge class and simplify helpers for unweighted BFS

BFS operates on unweighted graphs, so the cost field and weighted edge
helpers are unnecessary. Rename addUnweightedUndirectedEdge to
addUndirectedEdge.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Simplify reconstructPath: use LinkedList.addFirst to avoid reverse

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Fix Integer reference comparison bug in reconstructPath

Use intValue() instead of != to avoid reference comparison failure
for node indices >= 128 (outside Integer cache range).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* Refactor BridgesAdjacencyList: add docs, clean up code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Change findBridges return type from flat List<Integer> to List<int[]>

Each bridge is now returned as an int[2] pair instead of two consecutive
elements in a flat list. Updated both test files accordingly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Clarify findBridges Javadoc with concrete example

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Document parallel-edge limitation in BridgesAdjacencyList

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Remove BridgesAdjacencyListIterative and its test

Redundant with the recursive BridgesAdjacencyList implementation.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
@pull pull bot locked and limited conversation to collaborators Mar 16, 2026
@pull pull bot added the ⤵️ pull label Mar 16, 2026
@pull pull bot merged commit b6462bd into Uncodedtech:master Mar 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant