othmane/ohif-viewer
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Remove dead code; clean-up comments

Browse Source
This commit is contained in:
dannyrb 2020-06-09 13:39:17 -04:00
parent 3b81ebcc96
commit fab7334696
2 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
extensions/measurement-tracking/src/panels/PanelMeasurementTableTracking/index.js
View File

@ -46,7 +46,6 @@ function PanelMeasurementTableTracking({ servicesManager, commandsManager }) {
); );
setDisplayMeasurements(mappedMeasurements); setDisplayMeasurements(mappedMeasurements);
// eslint-ignore-next-line // eslint-ignore-next-line
}, [MeasurementService, trackedStudy, trackedSeries]);
}, [ }, [
MeasurementService, MeasurementService,
trackedStudy, trackedStudy,
@ -75,7 +74,6 @@ function PanelMeasurementTableTracking({ servicesManager, commandsManager }) {
} }
}, [displayStudySummary.key, trackedMeasurements, trackedStudy]); }, [displayStudySummary.key, trackedMeasurements, trackedStudy]);
// TODO: Listen for measurement service "adds" and updates
// TODO: Better way to consolidated, debounce, check on change? // TODO: Better way to consolidated, debounce, check on change?
// Are we exposing the right API for measurementService? // Are we exposing the right API for measurementService?
// This watches for ALL MeasurementService changes. It updates a timestamp, // This watches for ALL MeasurementService changes. It updates a timestamp,

2
extensions/measurement-tracking/src/panels/PanelStudyBrowserTracking/PanelStudyBrowserTracking.jsx
View File

@ -35,7 +35,7 @@ function PanelStudyBrowserTracking({
const [displaySets, setDisplaySets] = useState([]); const [displaySets, setDisplaySets] = useState([]);
const [thumbnailImageSrcMap, setThumbnailImageSrcMap] = 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(() => { useEffect(() => {
const { unsubscribe } = MeasurementService.subscribe( const { unsubscribe } = MeasurementService.subscribe(
MeasurementService.EVENTS.MEASUREMENT_ADDED, MeasurementService.EVENTS.MEASUREMENT_ADDED,