From db4b1829db89597fa1fa04f1e334b646a705a9c3 Mon Sep 17 00:00:00 2001 From: Abhinav Date: Thu, 12 Jan 2023 13:21:20 +0530 Subject: [PATCH] fix spike MemoryUsage logging --- src/utils/processStats.ts | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/utils/processStats.ts b/src/utils/processStats.ts index 64f900d58..80e3c404b 100644 --- a/src/utils/processStats.ts +++ b/src/utils/processStats.ts @@ -44,8 +44,8 @@ async function logSpikeMainMemoryUsage() { processMemoryInfo.private ); const previewMemoryUsage = Math.max( - previousMainProcessMemoryInfo.private, - previousMainProcessMemoryInfo.residentSet + previousMainProcessMemoryInfo.residentSet ?? 0, + previousMainProcessMemoryInfo.private ); const isSpiking = currentMemoryUsage - previewMemoryUsage >= @@ -92,9 +92,10 @@ let rendererUsingHighMemory = false; async function logSpikeRendererMemoryUsage() { const processMemoryInfo = await process.getProcessMemoryInfo(); const currentMemoryUsage = Math.max( - processMemoryInfo.residentSet, + processMemoryInfo.residentSet ?? 0, processMemoryInfo.private ); + const previewMemoryUsage = Math.max( previousRendererProcessMemoryInfo.private, previousRendererProcessMemoryInfo.residentSet @@ -110,6 +111,8 @@ async function logSpikeRendererMemoryUsage() { (isHighMemoryUsage && !rendererUsingHighMemory) || (!isHighMemoryUsage && rendererUsingHighMemory); + console.log(convertBytesToHumanReadable(currentMemoryUsage * 1024)); + if (isSpiking || shouldReport) { const normalizedCurrentProcessMemoryInfo = await getNormalizedProcessMemoryInfo(processMemoryInfo);