Two issues in set_e_repr()

[dmlc]

Two issues in set_e_repr()

[jermainewang]

I think this is a corner case that we didn't cover -- setting features when column does not exist. I will fix that in a patch. The edge list order is correctly captured. Here is the result on my PC:

>>> import networkx as nx
>>> import dgl
>>> import torch
>>> G = nx.DiGraph()
>>> G.add_nodes_from([0, 1, 2])
>>> G.add_edges_from([(0, 1), (0, 2)])
>>> G.edges
OutEdgeView([(0, 1), (0, 2)])
>>> G.nodes
NodeView((0, 1, 2))
>>> DG = dgl.DGLGraph(G)
>>> DG.edges
OutEdgeView([(0, 1), (0, 2)])
>>> DG.nodes
NodeView((0, 1, 2))
>>> DG.edge_list
[(0, 1), (0, 2)]

Setting features to non-exist column could be supported by padding zero. This has already been supported by FrameRef in the new subgraph branch (see this test file. However, the Graph class does not handle it correctly at the moment. Will fix this.

[jermainewang]

Such fix will be included in PR #39

[BarclayII]

Yes, switching to subgraph branch fixes problem (1).