From f4b92af8d975b888d206929d2d8a0f6b7ce89998 Mon Sep 17 00:00:00 2001
From: Igor Octaviano <igoroctaviano@gmail.com>
Date: Tue, 6 Oct 2026 08:35:24 -0300
Subject: [PATCH] chore(seg): remove verbose debug logs

Remove [SEG load] debug logs that were too verbose for production use.
---
 .../src/getSopClassHandlerModule.ts           | 51 +------------------
 1 file changed, 1 insertion(+), 50 deletions(-)

diff --git a/extensions/cornerstone-dicom-seg/src/getSopClassHandlerModule.ts b/extensions/cornerstone-dicom-seg/src/getSopClassHandlerModule.ts
index 88cd7a166b6..63b33028094 100644
--- a/extensions/cornerstone-dicom-seg/src/getSopClassHandlerModule.ts
+++ b/extensions/cornerstone-dicom-seg/src/getSopClassHandlerModule.ts
@@ -1,4 +1,4 @@
-import { utils, Types as OhifTypes, DicomMetadataStore, classes, log } from '@ohif/core';
+import { utils, Types as OhifTypes, DicomMetadataStore, classes } from '@ohif/core';
 import i18n from '@ohif/i18n';
 import { metaData, eventTarget, utilities as csUtils } from '@cornerstonejs/core';
 import { CONSTANTS, segmentation as cstSegmentation } from '@cornerstonejs/tools';
@@ -18,8 +18,6 @@ const LABELMAP_SEG_SOP_CLASS_UID = '1.2.840.10008.5.1.4.1.1.66.7';
 
 const loadPromises = {};
 
-const SEG_LOAD_LOG_PREFIX = '[SEG load]';
-
 // Max number of SEG frames fetched/decoded concurrently by the segmentation
 // loader. Hard-coded to 16 for now; intended to become configurable (and to
 // pair with the full-instance prefetch capability) in a follow-up.
@@ -203,35 +201,6 @@ function _resolveFrameImageIds(
   return frameImageIds.length ? frameImageIds : [segImageIdStr];
 }
 
-function _logSegImageIds({
-  segDisplaySet,
-  segImageIdStr,
-  frameImageIds,
-  referencedImageIds,
-}: {
-  segDisplaySet: AppTypes.DisplaySet;
-  segImageIdStr: string;
-  frameImageIds: string[];
-  referencedImageIds: string[];
-}) {
-  const instance = segDisplaySet.instance as Record<string, unknown>;
-  const numberOfFrames = Number(instance?.NumberOfFrames) || 1;
-
-  log.debug(SEG_LOAD_LOG_PREFIX, 'Loading SEG pixel data', {
-    SOPInstanceUID: segDisplaySet.SOPInstanceUID,
-    SeriesInstanceUID: segDisplaySet.SeriesInstanceUID,
-    SOPClassUID: segDisplaySet.SOPClassUID,
-    NumberOfFrames: numberOfFrames,
-    segmentCount: Object.keys(segDisplaySet.segments || {}).length,
-    referencedDisplaySetInstanceUID: segDisplaySet.referencedDisplaySetInstanceUID,
-    referencedImageIdCount: referencedImageIds.length,
-    referencedImageIds,
-    segImageIdForMetadata: segImageIdStr,
-    frameImageIds,
-    loadSegFramesIndividually: frameImageIds.length > 1,
-  });
-}
-
 function _getDisplaySetsFromSeries(
   instances,
   servicesManager: AppTypes.ServicesManager,
@@ -459,13 +428,6 @@ async function _loadSegments({
     ? stripFrameFromImageId(segImageIdStr)
     : segImageIdStr;
 
-  _logSegImageIds({
-    segDisplaySet,
-    segImageIdStr: segImageIdForMetadata,
-    frameImageIds,
-    referencedImageIds: imageIds,
-  });
-
   _ensureSegInstanceMetadataAvailable(segImageIdForMetadata, instance);
   frameImageIds.forEach(id => _ensureSegInstanceMetadataAvailable(id, instance));
 
@@ -557,17 +519,6 @@ async function _loadSegments({
   }
 
   Object.assign(segDisplaySet, results);
-
-  const labelMapImageIds = (results as { labelMapImages?: { imageId: string }[][] })
-    .labelMapImages?.flat()
-    .map(image => image.imageId);
-
-  log.debug(SEG_LOAD_LOG_PREFIX, 'SEG parse complete', {
-    SOPInstanceUID: segDisplaySet.SOPInstanceUID,
-    labelMapImageCount: labelMapImageIds?.length ?? 0,
-    labelMapImageIds,
-    segmentIndices: Object.keys(segDisplaySet.segments || {}),
-  });
 }
 
 function _segmentationExists(segDisplaySet) {