hdalsania commented on code in PR #993:
URL: https://github.com/apache/daffodil-vscode/pull/993#discussion_r1534119723
##########
src/svelte/src/components/ServerMetrics/ServerMetrics.svelte:
##########
@@ -107,32 +107,49 @@ limitations under the License.
on:mouseenter={() => showHeartbeatInfo(true)}
on:mouseleave={() => showHeartbeatInfo(false)}
>
- {#if heartbeat.latency < 20}
+ {#if 0 < heartbeat.latency && heartbeat.latency < 20}
<circle cx="50%" cy="50%" r="4pt" fill="green" />
- {:else if heartbeat.latency < 40}
+ {:else if 0 < heartbeat.latency && heartbeat.latency < 40}
<circle cx="50%" cy="50%" r="4pt" fill="yellow" />
- {:else if heartbeat.latency > 60}
+ {:else if 0 < heartbeat.latency && heartbeat.latency > 60}
<circle cx="50%" cy="50%" r="4pt" fill="red" />
{:else}
<circle cx="50%" cy="50%" r="4pt" fill="grey" />
{/if}
</svg>
<div class="heartbeat-info">
- <b>Latency:</b>
- {heartbeat.latency}ms,
- <b>CPU Load Avg:</b>
- {(heartbeat.serverCpuLoadAverage
- ? heartbeat.serverCpuLoadAverage
- : 0
- ).toFixed(2)},
- <b>Memory Usage:</b>
- {heartbeat.serverUsedMemory},
- <b>Session Count:</b>
- {heartbeat.sessionCount},
- <b>Uptime:</b>
- {prettyPrintUptime(heartbeat.serverUptime)}
- <b>JVM Version:</b>
- {heartbeat.jvmVersion}
+ {#if heartbeat.latency > 0}
+ <b>Latency:</b>
+ {heartbeat.latency}ms,
+ {/if}
+ {#if heartbeat.serverUptime > 0}
+ <b>Uptime:</b>
+ {prettyPrintUptime(heartbeat.serverUptime)}
+ {/if}
+ {#if heartbeat.sessionCount > 0}
+ <b>Session Count:</b>
+ {heartbeat.sessionCount},
+ {/if}
+ {#if heartbeat.serverCpuLoadAverage > 0}
Review Comment:
Because of this change, windows does not display CPU Load Avg. at all
because it is not >0?
Windows:
Linux:
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
