Skip to content

Preserve reciprocal edges in the NetworkX dialect - #88

Merged
j6k4m8 merged 1 commit into
masterfrom
fix/networkx-directed-edges-44
Jul 28, 2026
Merged

Preserve reciprocal edges in the NetworkX dialect#88
j6k4m8 merged 1 commit into
masterfrom
fix/networkx-directed-edges-44

Conversation

@j6k4m8

@j6k4m8 j6k4m8 commented Jul 28, 2026

Copy link
Copy Markdown
Member

Closes #44

  • use NetworkX OutEdgeView for directed backends and EdgeView for undirected backends
  • preserve reciprocal directed edges in iteration, calls, and metadata lookup
  • add regression coverage for both directions

Also I love that #88 closes #44, that feels somehow very good to me. Unfortunately #22 and #11 are totally unrelated.

@codspeed-hq

codspeed-hq Bot commented Jul 28, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 17 untouched benchmarks


Comparing fix/networkx-directed-edges-44 (97d793c) with master (f322db8)

Open in CodSpeed

@j6k4m8
j6k4m8 merged commit a9b7a52 into master Jul 28, 2026
7 checks passed
@j6k4m8
j6k4m8 deleted the fix/networkx-directed-edges-44 branch July 28, 2026 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NetworkXDialect does not work correctly with networkx.DiGraph

1 participant