From 44238de99abfb66ef6724df46a975b342273ee03 Mon Sep 17 00:00:00 2001 From: Lynn Shi Date: Wed, 14 Apr 2021 18:54:58 -0400 Subject: [PATCH] WIFI-2007 update based on review comments --- .../integration/utils/MqttStatsPublisher.java | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/opensync-ext-cloud/src/main/java/com/telecominfraproject/wlan/opensync/external/integration/utils/MqttStatsPublisher.java b/opensync-ext-cloud/src/main/java/com/telecominfraproject/wlan/opensync/external/integration/utils/MqttStatsPublisher.java index e25bc23..e67e915 100644 --- a/opensync-ext-cloud/src/main/java/com/telecominfraproject/wlan/opensync/external/integration/utils/MqttStatsPublisher.java +++ b/opensync-ext-cloud/src/main/java/com/telecominfraproject/wlan/opensync/external/integration/utils/MqttStatsPublisher.java @@ -1137,9 +1137,11 @@ public class MqttStatsPublisher { apNodeMetrics.setRadioUtilization(radioType, new ArrayList<>()); } apNodeMetrics.getRadioUtilization(radioType).add(radioUtil); - int noiseAvg = (int) Math.round(DecibelUtils.getAverageDecibel(toIntArray(noiseList))); - avgNoiseFloor.put(radioType, noiseAvg); - apNodeMetrics.setNoiseFloor(radioType, noiseAvg); + if (!noiseList.isEmpty()) { + int noiseAvg = (int) Math.round(DecibelUtils.getAverageDecibel(noiseList)); + avgNoiseFloor.put(radioType, noiseAvg); + apNodeMetrics.setNoiseFloor(radioType, noiseAvg); + } Long totalUtilization = Math.round((double) busy / totalDurationMs); Long totalNonWifi = totalUtilization - ((busyTx + busyRx) / totalDurationMs); @@ -1784,8 +1786,8 @@ public class MqttStatsPublisher { channelInfo.setTotalUtilization(totalUtilization.intValue()); channelInfo.setWifiUtilization(totalUtilization.intValue() - totalNonWifi.intValue()); channelInfo.setBandwidth(channelBandwidth); - if (noiseList.size() > 0) { - channelInfo.setNoiseFloor((int) Math.round(DecibelUtils.getAverageDecibel(toIntArray(noiseList)))); + if (!noiseList.isEmpty()) { + channelInfo.setNoiseFloor((int) Math.round(DecibelUtils.getAverageDecibel(noiseList))); } return channelInfo; }