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

update measurements panel when measurements change (debounced)

Browse Source
This commit is contained in:
dannyrb 2020-06-09 13:38:09 -04:00
parent f3e783b63e
commit 47f9e1391a
1 changed files with 37 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -2,6 +2,7 @@ import React, { useEffect, useState } from 'react';
import PropTypes from 'prop-types'; import PropTypes from 'prop-types';
import { StudySummary, MeasurementTable } from '@ohif/ui'; import { StudySummary, MeasurementTable } from '@ohif/ui';
import { DicomMetadataStore } from '@ohif/core'; import { DicomMetadataStore } from '@ohif/core';
import { useDebounce } from '@hooks';
import ActionButtons from './ActionButtons'; import ActionButtons from './ActionButtons';
import { useTrackedMeasurements } from '../../getContextModule'; import { useTrackedMeasurements } from '../../getContextModule';
@ -13,6 +14,13 @@ const DISPLAY_STUDY_SUMMARY_INITIAL_VALUE = {
}; };
function PanelMeasurementTableTracking({ servicesManager, commandsManager }) { function PanelMeasurementTableTracking({ servicesManager, commandsManager }) {
const [measurementChangeTimestamp, setMeasurementsUpdated] = useState(
Date.now().toString()
);
const debouncedMeasurementChangeTimestamp = useDebounce(
measurementChangeTimestamp,
200
);
const { MeasurementService } = servicesManager.services; const { MeasurementService } = servicesManager.services;
const [ const [
trackedMeasurements, trackedMeasurements,
@ -39,6 +47,12 @@ function PanelMeasurementTableTracking({ servicesManager, commandsManager }) {
setDisplayMeasurements(mappedMeasurements); setDisplayMeasurements(mappedMeasurements);
// eslint-ignore-next-line // eslint-ignore-next-line
}, [MeasurementService, trackedStudy, trackedSeries]); }, [MeasurementService, trackedStudy, trackedSeries]);
}, [
MeasurementService,
trackedStudy,
trackedSeries,
debouncedMeasurementChangeTimestamp,
]);
// ~~ DisplayStudySummary // ~~ DisplayStudySummary
useEffect(() => { useEffect(() => {
@ -62,23 +76,30 @@ function PanelMeasurementTableTracking({ servicesManager, commandsManager }) {
}, [displayStudySummary.key, trackedMeasurements, trackedStudy]); }, [displayStudySummary.key, trackedMeasurements, trackedStudy]);
// TODO: Listen for measurement service "adds" and updates // 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,
// which is debounced. After a brief period of inactivity, this triggers
// a re-render where we grab up-to-date measurements.
useEffect(() => { useEffect(() => {
// const { unsubscribe } = MeasurementService.subscribe( const added = MeasurementService.EVENTS.MEASUREMENT_ADDED;
// MeasurementService.EVENTS.MEASUREMENT_ADDED, const updated = MeasurementService.EVENTS.MEASUREMENT_UPDATED;
// ({ source, measurement }) => { const removed = MeasurementService.EVENTS.MEASUREMENT_REMOVED;
// const { const subscriptions = [];
// referenceSeriesUID: SeriesInstanceUID,
// referenceStudyUID: StudyInstanceUID, [added, updated, removed].forEach(evt => {
// } = measurement; subscriptions.push(
// sendTrackedMeasurementsEvent('TRACK_SERIES', { MeasurementService.subscribe(evt, () => {
// StudyInstanceUID, setMeasurementsUpdated(Date.now().toString());
// SeriesInstanceUID, })
// }); );
// console.log('PANEL:', measurement); });
// // console.log('Mapped:', annotation);
// } return () => {
// ); subscriptions.forEach(unsub => {
// return unsubscribe; unsub();
});
};
}, [MeasurementService, sendTrackedMeasurementsEvent]); }, [MeasurementService, sendTrackedMeasurementsEvent]);
const activeMeasurementItem = 0; const activeMeasurementItem = 0;