Conversation
…concepts, find_out_edge alias
# Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.
…d docs/status/coverage.md with per-file coverage breakdown\n- Link coverage report from docs/index.md\n- Add coverage metrics to docs/status/metrics.md\n- Generated from 4261 tests (100% pass) using linux-gcc-coverage preset
…atch dynamic_graph and compressed_graph - Swap VV,EV order to EV,VV in undirected_adjacency_list and all related classes - Update impl, api headers, tests, and documentation - Update incoming_edges_design.md with alias retention decision (Appendix D)
…vom/mom - Add voum_graph_traits: vector vertices + unordered_map edges (O(1) avg lookup) - Rename voem_graph_traits to vom_graph_traits (drop positional 'e') - Rename moem_graph_traits to mom_graph_traits (same convention) - Add voum tests and CPO edge map test coverage - Update all docs and counts from 26 to 27 trait combinations
All three value CPOs (edge_value, vertex_value, graph_value) use decltype(auto), which preserves the exact return type of the resolved member/ADL/default function: by-value (T), by-reference (T&), by-const-reference (const T&), and by-rvalue-reference (T&&). Updated: - edge_cpo.hpp: edge_value doc comment and @return - graph_cpo.hpp: vertex_value and graph_value doc comments and @return - cpo-reference.md: Value CPOs section with return-type rows - cpo-implementation.md: guideline #11 and checklist item"
- Add docs/assets/logo.svg (K4 graph in C++ blue/cyan, "GRAPH v3" text) - README.md: logo inline with title using HTML table layout - docs/index.md: logo inline with title using HTML table layout"
- Create agents/incoming_edges_plan.md with 10 self-contained phases - Second review pass on design doc: remove has_in_edge (duplicates contains_in_edge), add missing find_in_edge/contains_in_edge overloads, add pipe-syntax adaptor section (§8.4), expand Appendix A CPO table, fix trait wording, add topological_sort to §9 with precise call counts
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
1 participant
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.
A variety of changes to improve consistency and functionality.