docs(agents): restructure and correct bgl_migration_strategy.md#33
Merged
Conversation
Collaborator
pratzl
commented
May 31, 2026
- Fix factual contradictions: mutation support (dynamic_graph + undirected both have add/remove vertex/edge), transpose API (class not factory fn), scorecard mutation row (was 'undirected only / 40%')
- Add 'How to use this document' reader-orientation intro
- Restructure: move practical guide (§1-11) ahead of reference material; demote Algorithm Gap Analysis, Roadmap, and Scorecard to Appendices A/B/C
- Topological sort reclassified as traversal (with BFS/DFS) in Appendix A and Appendix C; traversal score updated to 100%
- Visitor row updated to reflect shipped composable adaptor toolkit (visitor_factory.hpp); score 75% -> 90%
- Property system score corrected to 100% (by design): tag-dispatch machinery was a C++03 workaround, not a missing feature
- Overall coverage re-evaluated: ~33% -> ~46%, with core/specialist split
- Appendix B: add Boost incubation strategy subsection (Boost as BGL2 to gain field usage for std::graph proposal; alias-layer packaging guidance)
- Fix proposal-voice in §8 (DOT writer): 'should leverage' -> 'leverages'
- MutableGraph concept note clarified: mutation exists as member functions
- Fix factual contradictions: mutation support (dynamic_graph + undirected both have add/remove vertex/edge), transpose API (class not factory fn), scorecard mutation row (was 'undirected only / 40%') - Add 'How to use this document' reader-orientation intro - Restructure: move practical guide (§1-11) ahead of reference material; demote Algorithm Gap Analysis, Roadmap, and Scorecard to Appendices A/B/C - Topological sort reclassified as traversal (with BFS/DFS) in Appendix A and Appendix C; traversal score updated to 100% - Visitor row updated to reflect shipped composable adaptor toolkit (visitor_factory.hpp); score 75% -> 90% - Property system score corrected to 100% (by design): tag-dispatch machinery was a C++03 workaround, not a missing feature - Overall coverage re-evaluated: ~33% -> ~46%, with core/specialist split - Appendix B: add Boost incubation strategy subsection (Boost as BGL2 to gain field usage for std::graph proposal; alias-layer packaging guidance) - Fix proposal-voice in §8 (DOT writer): 'should leverage' -> 'leverages' - MutableGraph concept note clarified: mutation exists as member functions
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.