fix: new maximum_adjacency_search interface, deprecating old broken one - #571
Merged
Merged
Conversation
|
Boost dependency footprint vs Header-inclusion weights (graph files pulling each direct dependency in):
Transitive Boost modules: 66 → 66 (0) |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! 🚀 New features to boost your workflow:
|
|
Compiler-warning counts vs
|
Becheler
force-pushed
the
fix/mas-297-dedup
branch
2 times, most recently
from
August 26, 2026 09:55
39c9f10 to
5ed6004
Compare
Becheler
force-pushed
the
fix/mas-297-dedup
branch
from
August 26, 2026 12:10
5ed6004 to
50bcd44
Compare
|
An automated preview of the documentation is available at https://571.graph.prtest3.cppalliance.org/libs/graph/doc/html/index.html If more commits are pushed to the pull request, the docs will rebuild at the same URL. 2026-08-26 17:15:29 UTC |
Becheler
force-pushed
the
fix/mas-297-dedup
branch
3 times, most recently
from
August 26, 2026 13:24
2c0c228 to
759a8ed
Compare
Becheler
force-pushed
the
fix/mas-297-dedup
branch
2 times, most recently
from
August 26, 2026 15:27
5e8ce47 to
14be0c6
Compare
Becheler
force-pushed
the
fix/mas-297-dedup
branch
from
August 26, 2026 17:12
14be0c6 to
69dd000
Compare
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
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.
#297
#286
Before submitting
developbranch.Type of change
Does this PR introduce a breaking change?
What this PR does
Updated documentation page is available here: https://571.graph.prtest3.cppalliance.org/libs/graph/doc/html/graph/algorithms/utility/maximum_adjacency_search.html
All the changes had to be brought in a single PR to be able to land on develop without having tests/coverage/documentation out of sync.
detail::mas_sweepfunctionboost::graphnamespace :mas_test_old.cppadding theBOOST_ALLOW_DEPRECATED_SYMBOLSto avoid CI warningsmast_test.cppwith modernized semantics.stoer_wagnerinternals to usedetail::mas_sweepwith a visitor: not doing it in this PR would have led to dead code indetail::mas_sweepwith failing test coverage.PR #569 solidified tests for both SW and MAS so we can be confident this PR does not break existing interfaces.
Motivation
The main problems in #286 and #297 still remained. This PR should fix all of them:
reach_count[t] += weight(u, t), matching the implementation, and the parameter tables document the weight map.stoer_wagner_phasenow callsmas_sweepwith a populated assignment set, so the loop is exercised.assignmentswas a dead (identity) parameter. The public interface is now the fully positionalboost::graph::maximum_adjacency_search(g, weights, vis, start, pq)with no assignments parameter, plus a convenience overload. The old named-parameter and assignment-map forms are kept but deprecated (removal planned for 1.95).Testing
Checklist
b2in thetest/directory).