error message : duplicate values while creating network

[almugabo]

error message : duplicate values while creating network

[vtraag]

Thanks!

Yes, perhaps we can explicitly mention the duplicate element that we encountered, we will look into it.

Note that the network is assumed to be undirected. In other words, if you have the edges listed as source node -> target node, then make sure that the reverse edge is not also present. For example, if the edge 3, 4 is included, the edge 4, 3 should not be included. Could you check whether removing such duplicates resolves the issue?

[almugabo]

Thanks for the prompt reply.
Yes the "reciprocal" edges were the problems. When removed everything works fine.
Explicitly mentioning the "offending" can help, especially if the input network is large, find them can be time consuming.

[vtraag]

Good to hear it is solved!

Yes, I can understand, we'll look into improving the error message. I'll leave the issue open for now.

[Marmeladenbrot]

Are directed networks (and therefore multiple edges for the same nodes) possible in the algorithm? Or will it stay undirected network only?

[vtraag]

Multiple edges are not possible in this implementation. However, multiple edges can simply be represented as a single weighted edge, with the weight the multiplicity of the edges (or the sum of the weights).

This implementation accepts only undirected networks. Note that for CPM the direction is not important, while for modularity directionality may play a role. The Python implementation does allow for directed networks with multiple edges though.

[FlashZdy]

Does nodes must be continual and start from 0 ?

[vtraag]

Yes

[vtraag]

This is now fixed in the master branch. We now provide an explicit line numbers on errors, see commit 44829f1