opeltre / topos

Statistics and Topology
MIT License
9 stars 0 forks source link

Sort graph hyperedges #37

Open opeltre opened 2 years ago

opeltre commented 2 years ago

The effect of the sort=True kwarg has been removed by some commit.

A lexicographic sorting functionality on hyperedges should still be optional, either by kwarg or by a classmethod (probably cleaner).

This may right now lead to inconsistent behaviour (e.g. IsingNetwork.lift_node_beliefs) as calling Nerve.classify will sort hyperedges lexicographically. Maybe working with the base Graph.Index methods still makes it possible to keep indices in order...

Sort this out!