fix(capelin): Eliminate unnecessary double to long conversions

This change eliminates the unnecessary conversions from double to long
in the Capelin metric processing code.

1 files changed, 30 insertions, 30 deletions

diff --git a/opendc-experiments/opendc-experiments-capelin/src/test/kotlin/org/opendc/experiments/capelin/CapelinIntegrationTest.kt b/opendc-experiments/opendc-experiments-capelin/src/test/kotlin/org/opendc/experiments/capelin/CapelinIntegrationTest.kt
index 9b98b329..8008c944 100644
--- a/opendc-experiments/opendc-experiments-capelin/src/test/kotlin/org/opendc/experiments/capelin/CapelinIntegrationTest.kt
+++ b/opendc-experiments/opendc-experiments-capelin/src/test/kotlin/org/opendc/experiments/capelin/CapelinIntegrationTest.kt
@@ -120,10 +120,10 @@ class CapelinIntegrationTest {
             { assertEquals(0, monitorResults.runningVms, "All VMs should finish after a run") },
             { assertEquals(0, monitorResults.unscheduledVms, "No VM should not be unscheduled") },
             { assertEquals(0, monitorResults.queuedVms, "No VM should not be in the queue") },
-            { assertEquals(220346369672, monitor.totalRequestedBurst) { "Incorrect requested burst" } },
-            { assertEquals(206667809431, monitor.totalGrantedBurst) { "Incorrect granted burst" } },
-            { assertEquals(1151611104, monitor.totalOvercommissionedBurst) { "Incorrect overcommitted burst" } },
-            { assertEquals(0, monitor.totalInterferedBurst) { "Incorrect interfered burst" } }
+            { assertEquals(220346369753, monitor.totalWork) { "Incorrect requested burst" } },
+            { assertEquals(206667809529, monitor.totalGrantedWork) { "Incorrect granted burst" } },
+            { assertEquals(1151611104, monitor.totalOvercommittedWork) { "Incorrect overcommitted burst" } },
+            { assertEquals(0, monitor.totalInterferedWork) { "Incorrect interfered burst" } }
         )
     }
 
@@ -160,10 +160,10 @@ class CapelinIntegrationTest {
 
         // Note that these values have been verified beforehand
         assertAll(
-            { assertEquals(38051879542, monitor.totalRequestedBurst) { "Total requested work incorrect" } },
-            { assertEquals(34888186396, monitor.totalGrantedBurst) { "Total granted work incorrect" } },
-            { assertEquals(971668973, monitor.totalOvercommissionedBurst) { "Total overcommitted work incorrect" } },
-            { assertEquals(0, monitor.totalInterferedBurst) { "Total interfered work incorrect" } }
+            { assertEquals(38051879552, monitor.totalWork) { "Total requested work incorrect" } },
+            { assertEquals(34888186408, monitor.totalGrantedWork) { "Total granted work incorrect" } },
+            { assertEquals(971668973, monitor.totalOvercommittedWork) { "Total overcommitted work incorrect" } },
+            { assertEquals(0, monitor.totalInterferedWork) { "Total interfered work incorrect" } }
         )
     }
 
@@ -204,10 +204,10 @@ class CapelinIntegrationTest {
 
         // Note that these values have been verified beforehand
         assertAll(
-            { assertEquals(38051879542, monitor.totalRequestedBurst) { "Total requested work incorrect" } },
-            { assertEquals(34888186396, monitor.totalGrantedBurst) { "Total granted work incorrect" } },
-            { assertEquals(971668973, monitor.totalOvercommissionedBurst) { "Total overcommitted work incorrect" } },
-            { assertEquals(13910799, monitor.totalInterferedBurst) { "Total interfered work incorrect" } }
+            { assertEquals(38051879552, monitor.totalWork) { "Total requested work incorrect" } },
+            { assertEquals(34888186408, monitor.totalGrantedWork) { "Total granted work incorrect" } },
+            { assertEquals(971668973, monitor.totalOvercommittedWork) { "Total overcommitted work incorrect" } },
+            { assertEquals(13910814, monitor.totalInterferedWork) { "Total interfered work incorrect" } }
         )
     }
 
@@ -256,10 +256,10 @@ class CapelinIntegrationTest {
 
         // Note that these values have been verified beforehand
         assertAll(
-            { assertEquals(25412073100, monitor.totalRequestedBurst) { "Total requested work incorrect" } },
-            { assertEquals(23695061847, monitor.totalGrantedBurst) { "Total granted work incorrect" } },
-            { assertEquals(368502468, monitor.totalOvercommissionedBurst) { "Total overcommitted work incorrect" } },
-            { assertEquals(0, monitor.totalInterferedBurst) { "Total interfered work incorrect" } }
+            { assertEquals(25412073109, monitor.totalWork) { "Total requested work incorrect" } },
+            { assertEquals(23695061858, monitor.totalGrantedWork) { "Total granted work incorrect" } },
+            { assertEquals(368502468, monitor.totalOvercommittedWork) { "Total overcommitted work incorrect" } },
+            { assertEquals(0, monitor.totalInterferedWork) { "Total interfered work incorrect" } }
         )
     }
 
@@ -283,27 +283,27 @@ class CapelinIntegrationTest {
     }
 
     class TestExperimentReporter : ExperimentMonitor {
-        var totalRequestedBurst = 0L
-        var totalGrantedBurst = 0L
-        var totalOvercommissionedBurst = 0L
-        var totalInterferedBurst = 0L
+        var totalWork = 0L
+        var totalGrantedWork = 0L
+        var totalOvercommittedWork = 0L
+        var totalInterferedWork = 0L
 
-        override fun reportHostSlice(
+        override fun reportHostData(
             time: Long,
-            requestedBurst: Long,
-            grantedBurst: Long,
-            overcommissionedBurst: Long,
-            interferedBurst: Long,
+            totalWork: Double,
+            grantedWork: Double,
+            overcommittedWork: Double,
+            interferedWork: Double,
             cpuUsage: Double,
             cpuDemand: Double,
             powerDraw: Double,
-            numberOfDeployedImages: Int,
+            instanceCount: Int,
             host: Host,
         ) {
-            totalRequestedBurst += requestedBurst.toLong()
-            totalGrantedBurst += grantedBurst.toLong()
-            totalOvercommissionedBurst += overcommissionedBurst.toLong()
-            totalInterferedBurst += interferedBurst.toLong()
+            this.totalWork += totalWork.toLong()
+            totalGrantedWork += grantedWork.toLong()
+            totalOvercommittedWork += overcommittedWork.toLong()
+            totalInterferedWork += interferedWork.toLong()
         }
 
         override fun close() {}