Remove node values of the branch in setitem - #30

Draft

nvaytet wants to merge 33 commits into

mainfrom

node-values-fixes

Commits on Aug 20, 2025

Groupby notes and ideas
SimonHeybrock
committed
1f2511a
More details
SimonHeybrock
committed
a3ffc8c
More thoughts and problems
SimonHeybrock
committed
a226c14
Split large file
SimonHeybrock
committed
6501bc3
Try supporting some MultiIndex series
SimonHeybrock
committed
28322c5
Begin adding groupby
SimonHeybrock
committed
b8a81ee
Confirm that current to_networkx does not group correctly
SimonHeybrock
committed
dd745af
Extract helper method
SimonHeybrock
committed
25cf5b2
First version passing tests
SimonHeybrock
committed
d4f23c9
Fewer hacks?
SimonHeybrock
committed
1426e96
Write down some better ideas
SimonHeybrock
committed
7ada209
Refactor again
SimonHeybrock
committed
b20ee10
Refactor again
SimonHeybrock
committed
5f71d1e
Some cleanup
SimonHeybrock
committed
8e9dae8
Rename variables
SimonHeybrock
committed
6229ebe
Revert MultiIndex support
SimonHeybrock
committed
fd4e32a
Notes
SimonHeybrock
committed
1d852ed
Add TODO
SimonHeybrock
committed
6e125a8
Sketch out cleaner solution
SimonHeybrock
committed
366b43f
Sketch out cleaner approach
SimonHeybrock
committed
75283ad
Cleanup
SimonHeybrock
committed
14b663d
More cleanup
SimonHeybrock
committed
40331bc
Cleanup test
SimonHeybrock
committed
fd43c92
Check that multiple groupings in same graph currently not handled
SimonHeybrock
committed
a81600b
Hack that works in more cases, but not recursive groupby (yet?)
SimonHeybrock
committed
6226448

Commits on Aug 21, 2025

Clone grouping links separately, should work in more complex cases
SimonHeybrock
committed
5507f23
Cleanup
SimonHeybrock
committed
4c7f5e2
Test mapped node data is preserved in setitem
SimonHeybrock
committed
0b77605
Update tests
SimonHeybrock
committed
42a1493
Test grouping twice
SimonHeybrock
committed
91cc000
Explain
SimonHeybrock
committed
082d586

Commits on Sep 8, 2025