reticula-network / reticula-python Goto Github PK

reticula.from_networkx[ret.int64] is trying to delegate to reticula._from_networkx__int64() which has an extra underscore.

The binding for these two members are missing.

This is useful as residual for delta distribution.

The implicit_event_graph, component, component_size and component_size_estimation classes do not have documentation for their methods.

The representation for undirected_hyperedge and undirected_temporal_hyperedge should include brackets around the list of vertices.

>>> ret.undirected_temporal_hyperedge[ret.int64, ret.double]([3, 4, 5, 6, 7], 0.2)
undirected_temporal_hyperedge[int64, double](3, 4, 5, 6, 7, t=0.2)  # should be ([3, 4, 5, 6, 7], time=0.2)
>>> ret.undirected_hyperedge[ret.int64]([3, 4, 5, 6, 7])
undirected_hyperedge[int64](3, 4, 5, 6, 7) # should be ([3, 4, 5, 6, 7])

NotImplementedError feels like the wrong kind of exception for what is trying to be conveyed here.

Currently copying a network is a hassle. A simple binding for the copy constructor would solve this issue for many types.