Fixed problem with degree distribution chart.

It was was not getting reset when the graph was updated. The issue only affected the visualisation- the underlying calculations are correct (as illustrated by the newly added unit test) Closes #44.
mbatchkarov · Jul 3, 2014 · 2b404ba · 2b404ba
1 parent 84ad088
commit 2b404ba
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/main/java/controller/Stats.java b/src/main/java/controller/Stats.java
@@ -530,6 +530,7 @@ public void updateDegreeDistributionChartData(boolean cumulative) {
         } else {
             buckets = getCumulativeDegreeDistribution();
         }
+        this.degreeDistXYSeries.clear();
         for (int i = 0; i < buckets.length; i++) {
             if (buckets[i] > 0) {
                 if (buckets[i] > 0) {

diff --git a/src/test/java/controller/StatsTest.java b/src/test/java/controller/StatsTest.java
@@ -121,4 +121,16 @@ public void testGetAvgDegree() throws Exception {
         glass.addEdge(e(4), v(1), v(3));
         assertEquals(2.5, glassStats.getAvgDegree(), 0.01);
     }
+
+    @Test
+    public void testGetDegreDistribution(){
+
+        Stats s = new Stats(triangle);
+        assertArrayEquals(s.getDegreeDistribution(),
+                          new int[]{0, 0, 3}); // three vertices of degree 2
+
+        s = new Stats(glass);
+        assertArrayEquals(s.getDegreeDistribution(),
+                          new int[]{0, 1, 2, 1});
+    }
 }