Small performance improvement #16

Consider multiple edges between a node pair as one edge
cytoscape · Oct 5, 2017 · ed8c0a4 · ed8c0a4
1 parent bdba224
commit ed8c0a4
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/cytoscape-cose-bilkent.js b/cytoscape-cose-bilkent.js
diff --git a/src/Layout/index.js b/src/Layout/index.js
@@ -157,8 +157,10 @@ _CoSELayout.prototype.run = function () {
     var edge = edges[i];
     var sourceNode = this.idToLNode[edge.data("source")];
     var targetNode = this.idToLNode[edge.data("target")];
-    var e1 = gm.add(layout.newEdge(), sourceNode, targetNode);
-    e1.id = edge.id();
+    if(sourceNode.getEdgesBetween(targetNode).length == 0){
+      var e1 = gm.add(layout.newEdge(), sourceNode, targetNode);
+      e1.id = edge.id();
+    }
   }
 
    var getPositions = function(ele, i){