Data Laboratory: wrong edge type created (mutual instead of directed)

Bug #787401 reported by Sébastien Heymann on 2011-05-24
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Gephi
Fix Released
Critical
Eduardo Ramos

Bug Description

Way to reproduce:

1. Open a new project, and go in the data lab.
2. Create the nodes A and B
3. Create an undirected edge between A and B
4. Create a node C
5. Create a directed edge from A to C.
6. Go in the Preview

You'll see that the edge (A,C) is not directed but mutual, however it is shown as directed in the Data Lab.
If the edge between A and B is directed, there is no error.

Changed in gephi:
assignee: nobody → Eduardo Ramos (eduramiba)
milestone: none → 0.8alpha
importance: Undecided → Critical
tags: added: datalab
Eduardo Ramos (eduramiba) wrote :

Fixed in rev 2231

Hi, it seems the real problem is in preview visualization. This bug only happens with mixed graphs.
In the class PreviewGraphFactory, getEdge(node1,node2) method of mixed graph is used to find if a directed edge exists in the opposite direction of a given edge, to show it as bidirectional.

The behaviour of getEdge for a mixed graph is not clear in the javadoc, but I assume it first tries to return a node with node1 as source and node2 as target but if not found then tries to return the opposite (and checking the code I think this is correct).

So checking that the direction of the returned edge really means it is bidirectional, this graph and other graphs show correctly.

Eduardo

Changed in gephi:
status: New → Fix Committed
Changed in gephi:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers