Removes calls to visualStyle.apply() and view.updateView(), because they are redundant and could make style updates slower. Also don’t set the EM style to the view again (when it’s already set to that view).

Author: chrtannus

EnrichmentMapPlugin/src/main/java/org/baderlab/csplugins/enrichmentmap/task/ApplyEMStyleTask.java

@@ -47,13 +47,10 @@ public void run(TaskMonitor taskMonitor) throws Exception {
 	private void applyVisualStyle() {
 		CyNetworkView view = options.getNetworkView();
 		VisualStyle vs = getVisualStyle(options.getEnrichmentMap());
+		styleBuilderProvider.get().updateProperties(vs, options, chart);
 
-		EMStyleBuilder styleBuilder = styleBuilderProvider.get();
-		styleBuilder.updateProperties(vs, options, chart);
-
-		visualMappingManager.setVisualStyle(vs, view);
-		vs.apply(view);
-		view.updateView();
+		if (!vs.equals(visualMappingManager.getVisualStyle(view)))
+			visualMappingManager.setVisualStyle(vs, view);
 	}
 
 	private VisualStyle getVisualStyle(EnrichmentMap map) {