From fab7334696bc998840eadf2f09ab33c18ebf89b9 Mon Sep 17 00:00:00 2001 From: dannyrb Date: Tue, 9 Jun 2020 13:39:17 -0400 Subject: [PATCH] Remove dead code; clean-up comments --- .../src/panels/PanelMeasurementTableTracking/index.js | 2 -- .../PanelStudyBrowserTracking/PanelStudyBrowserTracking.jsx | 2 +- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/extensions/measurement-tracking/src/panels/PanelMeasurementTableTracking/index.js b/extensions/measurement-tracking/src/panels/PanelMeasurementTableTracking/index.js index a0d57f430..8cf58829e 100644 --- a/extensions/measurement-tracking/src/panels/PanelMeasurementTableTracking/index.js +++ b/extensions/measurement-tracking/src/panels/PanelMeasurementTableTracking/index.js @@ -46,7 +46,6 @@ function PanelMeasurementTableTracking({ servicesManager, commandsManager }) { ); setDisplayMeasurements(mappedMeasurements); // eslint-ignore-next-line - }, [MeasurementService, trackedStudy, trackedSeries]); }, [ MeasurementService, trackedStudy, @@ -75,7 +74,6 @@ function PanelMeasurementTableTracking({ servicesManager, commandsManager }) { } }, [displayStudySummary.key, trackedMeasurements, trackedStudy]); - // TODO: Listen for measurement service "adds" and updates // TODO: Better way to consolidated, debounce, check on change? // Are we exposing the right API for measurementService? // This watches for ALL MeasurementService changes. It updates a timestamp, diff --git a/extensions/measurement-tracking/src/panels/PanelStudyBrowserTracking/PanelStudyBrowserTracking.jsx b/extensions/measurement-tracking/src/panels/PanelStudyBrowserTracking/PanelStudyBrowserTracking.jsx index 5184795a6..ea2e4441b 100644 --- a/extensions/measurement-tracking/src/panels/PanelStudyBrowserTracking/PanelStudyBrowserTracking.jsx +++ b/extensions/measurement-tracking/src/panels/PanelStudyBrowserTracking/PanelStudyBrowserTracking.jsx @@ -35,7 +35,7 @@ function PanelStudyBrowserTracking({ const [displaySets, setDisplaySets] = useState([]); const [thumbnailImageSrcMap, setThumbnailImageSrcMap] = useState({}); - // TODO: Listen for measurement service "adds" (really shouldn't be added until cornerstone-tools "complete") + // TODO: Should this be somewhere else? Feels more like a mode "lifecycle" setup/destroy? useEffect(() => { const { unsubscribe } = MeasurementService.subscribe( MeasurementService.EVENTS.MEASUREMENT_ADDED,