WIP

extensions/dicom-sr/src/OHIFCornerstoneSRViewport.js

@@ -1,27 +1,12 @@
-import React, { Component } from 'react';
+import React, { useEffect, useState } from 'react';
-import CornerstoneViewport from 'react-cornerstone-viewport';
-//import ConnectedCornerstoneViewport from './ConnectedCornerstoneViewport';
-import OHIF from '@ohif/core';
 import PropTypes from 'prop-types';
-import cornerstone from 'cornerstone-core';
+import cornerstoneTools from 'cornerstone-tools';
-import debounce from 'lodash.debounce';
-import throttle from 'lodash.throttle';
 
-// const {
+import CornerstoneViewport from 'react-cornerstone-viewport';
-//   onAdded,
+import OHIF from '@ohif/core';
-//   onRemoved,
+import { ViewportActionBar, useViewportGrid } from '@ohif/ui';
-//   onModified,
+import TOOL_NAMES from './constants/toolNames';
-// } = OHIF.measurements.MeasurementHandlers;
+import id from './id';
-
-// // TODO: Transition to enums for the action names so that we can ensure they stay up to date
-// // everywhere they're used.
-// const MEASUREMENT_ACTION_MAP = {
-//   added: onAdded,
-//   removed: onRemoved,
-//   modified: throttle(event => {
-//     return onModified(event);
-//   }, 300),
-// };
 
 // const cine = viewportSpecificData.cine;
 
@@ -30,96 +15,53 @@ import throttle from 'lodash.throttle';
 
 const { StackManager } = OHIF.utils;
 
-class OHIFCornerstoneViewport extends Component {
+function OHIFCornerstoneSRViewport({
-  state = {
+  children,
-    viewportData: null,
+  dataSource,
-  };
+  displaySet,
+  viewportIndex,
+  DisplaySetService,
+}) {
+  const [viewportGrid, dispatchViewportGrid] = useViewportGrid();
+  const [measurementSelected, setMeasurementSelected] = useState(0);
+  const [measurementCount, setMeasurementCount] = useState(1);
+  const [viewportData, setViewportData] = useState(null);
+  const [activeDisplaySetData, setActiveDisplaySetData] = useState({});
+  const [element, setElement] = useState(null);
 
-  static defaultProps = {
+  const { viewports } = viewportGrid;
-    customProps: {},
-  };
 
-  static propTypes = {
+  const onElementEnabled = evt => {
-    displaySet: PropTypes.object,
+    const eventData = evt.detail;
-    viewportIndex: PropTypes.number,
+    const { element } = eventData;
-    dataSource: PropTypes.object,
-    children: PropTypes.node,
-    customProps: PropTypes.object,
-  };
 
-  static name = 'OHIFCornerstoneViewport';
+    const { measurements } = displaySet;
 
-  static init() {
+    const srModule = cornerstoneTools.getModule(id);
-    console.log('OHIFCornerstoneViewport init()');
-  }
 
-  static destroy() {
+    srModule.setters.trackingUniqueIdentifiersForElement(
-    console.log('OHIFCornerstoneViewport destroy()');
+      element,
-    StackManager.clearStacks();
+      measurements.map(measurement => measurement.TrackingUniqueIdentifier),
-  }
+      measurementSelected
-
-  /**
-   * Obtain the CornerstoneTools Stack for the specified display set.
-   *
-   * @param {Object} displaySet
-   * @param {Object} dataSource
-   * @return {Object} CornerstoneTools Stack
-   */
-  static getCornerstoneStack(displaySet, dataSource) {
-    const { frameIndex } = displaySet;
-
-    // Get stack from Stack Manager
-    const storedStack = StackManager.findOrCreateStack(displaySet, dataSource);
-
-    // Clone the stack here so we don't mutate it
-    const stack = Object.assign({}, storedStack);
-
-    stack.currentImageIdIndex = frameIndex;
-
-    // TODO -> Do we ever use this like this?
-    // if (SOPInstanceUID) {
-    //   const index = stack.imageIds.findIndex(imageId => {
-    //     const imageIdSOPInstanceUID = cornerstone.metaData.get(
-    //       'SOPInstanceUID',
-    //       imageId
-    //     );
-
-    //     return imageIdSOPInstanceUID === SOPInstanceUID;
-    //   });
-
-    //   if (index > -1) {
-    //     stack.currentImageIdIndex = index;
-    //   } else {
-    //     console.warn(
-    //       'SOPInstanceUID provided was not found in specified DisplaySet'
-    //     );
-    //   }
-    // }
-
-    return stack;
-  }
-
-  getViewportData = async displaySet => {
-    let viewportData;
-
-    const { dataSource } = this.props;
-
-    const stack = OHIFCornerstoneViewport.getCornerstoneStack(
-      displaySet,
-      dataSource
     );
 
-    viewportData = {
+    setElement(element);
-      StudyInstanceUID: displaySet.StudyInstanceUID,
-      displaySetInstanceUID: displaySet.displaySetInstanceUID,
-      stack,
-    };
-
-    return viewportData;
   };
 
-  setStateFromProps() {
+  useEffect(() => {
-    const { displaySet } = this.props;
+    const numMeasurements = displaySet.measurements.length;
+
+    console.log(`MEASUREMENT COUNT: ${numMeasurements}`);
+
+    setMeasurementCount(numMeasurements);
+  }, [
+    dataSource,
+    displaySet,
+    displaySet.StudyInstanceUID,
+    displaySet.displaySetInstanceUID,
+  ]);
+
+  const updateViewport = () => {
     const {
       StudyInstanceUID,
       displaySetInstanceUID,
@@ -136,97 +78,219 @@ class OHIFCornerstoneViewport extends Component {
       );
     }
 
-    this.getViewportData(displaySet).then(viewportData => {
+    console.log(measurementSelected);
-      this.setState({
+
-        viewportData,
+    _getViewportData(
-      });
+      dataSource,
+      displaySet,
+      measurementSelected,
+      DisplaySetService,
+      element
+    ).then(viewportData => {
+      setViewportData({ ...viewportData });
+    });
+  };
+
+  useEffect(() => {
+    updateViewport();
+  }, [
+    dataSource,
+    displaySet,
+    displaySet.StudyInstanceUID,
+    displaySet.displaySetInstanceUID,
+    measurementSelected,
+  ]);
+
+  // ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+  let childrenWithProps = null;
+
+  if (!viewportData) {
+    return null;
+  }
+
+  const {
+    imageIds,
+    currentImageIdIndex,
+    // If this comes from the instance, would be a better default
+    // `FrameTime` in the instance
+    // frameRate = 0,
+  } = viewportData.stack;
+
+  // TODO: Does it make more sense to use Context?
+  if (children && children.length) {
+    childrenWithProps = children.map((child, index) => {
+      return (
+        child &&
+        React.cloneElement(child, {
+          viewportIndex,
+          key: index,
+        })
+      );
     });
   }
 
-  componentDidMount() {
+  const {
-    this.setStateFromProps();
+    Modality,
-  }
+    SeriesDate,
+    SeriesDescription,
+    SeriesInstanceUID,
+    SeriesNumber,
+  } = displaySet;
 
-  componentDidUpdate(prevProps) {
+  // TODO -> Get this from the associated stack.
-    const { displaySet } = this.props;
-    const prevDisplaySet = prevProps.displaySet;
 
-    if (
+  const {
-      displaySet.displaySetInstanceUID !==
+    PatientID,
-        prevDisplaySet.displaySetInstanceUID ||
+    PatientName,
-      displaySet.SOPInstanceUID !== prevDisplaySet.SOPInstanceUID ||
+    PatientSex,
-      displaySet.frameIndex !== prevDisplaySet.frameIndex
+    PatientAge,
-    ) {
+    SliceThickness,
-      this.setStateFromProps();
+  } = activeDisplaySetData;
-    }
-  }
 
-  render() {
+  const onMeasurementChange = direction => {
-    let childrenWithProps = null;
+    let newMeausrementSelected = measurementSelected;
 
-    if (!this.state.viewportData) {
+    if (direction === 'right') {
-      return null;
+      newMeausrementSelected++;
-    }
-    const { viewportIndex } = this.props;
-    const {
-      imageIds,
-      currentImageIdIndex,
-      // If this comes from the instance, would be a better default
-      // `FrameTime` in the instance
-      // frameRate = 0,
-    } = this.state.viewportData.stack;
 
-    // TODO: Does it make more sense to use Context?
+      if (newMeausrementSelected >= measurementCount) {
-    if (this.props.children && this.props.children.length) {
+        newMeausrementSelected = 0;
-      childrenWithProps = this.props.children.map((child, index) => {
+      }
-        return (
+    } else {
-          child &&
+      newMeausrementSelected--;
-          React.cloneElement(child, {
+
-            viewportIndex: this.props.viewportIndex,
+      if (newMeausrementSelected < 0) {
-            key: index,
+        newMeausrementSelected = measurementCount - 1;
-          })
+      }
-        );
-      });
     }
 
-    const debouncedNewImageHandler = debounce(
+    if (newMeausrementSelected === measurementSelected) {
-      ({ currentImageIdIndex, sopInstanceUid }) => {
+      updateViewport();
-        const { displaySet } = this.props;
+    }
-        const { StudyInstanceUID } = displaySet;
-        if (currentImageIdIndex > 0) {
-          this.props.onNewImage({
-            StudyInstanceUID,
-            SOPInstanceUID: sopInstanceUid,
-            frameIndex: currentImageIdIndex,
-            activeViewportIndex: viewportIndex,
-          });
-        }
-      },
-      700
-    );
 
-    // TODO -> We may still want a wrapped component to define all the measurement api stuff.
+    setMeasurementSelected(newMeausrementSelected);
+  };
 
-    return (
+  console.log(currentImageIdIndex);
-      <>
+
-        <CornerstoneViewport
+  return (
-          viewportIndex={viewportIndex}
+    <>
-          imageIds={imageIds}
+      <ViewportActionBar
-          imageIdIndex={currentImageIdIndex}
+        onSeriesChange={onMeasurementChange}
-          onNewImage={debouncedNewImageHandler}
+        studyData={{
-          // TODO: ViewportGrid Context?
+          label: '',
-          isActive={true} // todo
+          isTracked: false,
-          isStackPrefetchEnabled={true} // todo
+          isLocked: false,
-          isPlaying={false}
+          studyDate: SeriesDate, // TODO: This is series date. Is that ok?
-          frameRate={24}
+          currentSeries: SeriesNumber,
-        />
+          seriesDescription: SeriesDescription,
-        {childrenWithProps}
+          modality: Modality,
-      </>
+          patientInformation: {
-    );
+            patientName: PatientName ? PatientName.Alphabetic || '' : '',
-  }
+            patientSex: PatientSex || '',
+            patientAge: PatientAge || '',
+            MRN: PatientID || '',
+            thickness: `${SliceThickness}mm`,
+            spacing: '',
+            scanner: '',
+          },
+        }}
+      />
+      <CornerstoneViewport
+        onElementEnabled={onElementEnabled}
+        viewportIndex={viewportIndex}
+        imageIds={imageIds}
+        imageIdIndex={currentImageIdIndex}
+        // TODO: ViewportGrid Context?
+        isActive={true} // todo
+        isStackPrefetchEnabled={true} // todo
+        isPlaying={false}
+        frameRate={24}
+      />
+      {childrenWithProps}
+    </>
+  );
 }
 
-const temp = () => <div>Hello SR Viewport!</div>;
+OHIFCornerstoneSRViewport.propTypes = {
+  displaySet: PropTypes.object.isRequired,
+  viewportIndex: PropTypes.number.isRequired,
+  dataSource: PropTypes.object,
+  children: PropTypes.node,
+  customProps: PropTypes.object,
+};
 
-//export default OHIFCornerstoneViewport;
+OHIFCornerstoneSRViewport.defaultProps = {
-export default temp;
+  customProps: {},
+};
+
+/**
+ * Obtain the CornerstoneTools Stack for the specified display set.
+ *
+ * @param {Object} displaySet
+ * @param {Object} dataSource
+ * @return {Object} CornerstoneTools Stack
+ */
+function _getCornerstoneStack(
+  measurement,
+  dataSource,
+  DisplaySetService,
+  element
+) {
+  const { displaySetInstanceUID, TrackingUniqueIdentifier } = measurement;
+
+  const displaySet = DisplaySetService.getDisplaySetByUID(
+    displaySetInstanceUID
+  );
+
+  // Get stack from Stack Manager
+  const storedStack = StackManager.findOrCreateStack(displaySet, dataSource);
+
+  // Clone the stack here so we don't mutate it
+  const stack = Object.assign({}, storedStack);
+
+  const { imageId } = measurement;
+
+  stack.currentImageIdIndex = stack.imageIds.findIndex(i => i === imageId);
+
+  if (element) {
+    const srModule = cornerstoneTools.getModule(id);
+
+    srModule.setters.activeTrackingUniqueIdentifierForElement(
+      element,
+      TrackingUniqueIdentifier
+    );
+  }
+
+  return stack;
+}
+
+async function _getViewportData(
+  dataSource,
+  displaySet,
+  measurementSelected,
+  DisplaySetService,
+  element
+) {
+  let viewportData;
+
+  const { measurements } = displaySet;
+  const measurement = measurements[measurementSelected];
+
+  const stack = _getCornerstoneStack(
+    measurement,
+    dataSource,
+    DisplaySetService,
+    element
+  );
+
+  viewportData = {
+    StudyInstanceUID: displaySet.StudyInstanceUID,
+    displaySetInstanceUID: displaySet.displaySetInstanceUID,
+    stack,
+  };
+
+  return viewportData;
+}
+
+export default OHIFCornerstoneSRViewport;

extensions/dicom-sr/src/getSopClassHandlerModule.js

@@ -78,6 +78,7 @@ function _getDisplaySetsFromSeries(
     SOPClassHandlerId: `${id}.sopClassHandlerModule.${sopClassHandlerName}`,
     referencedImages: _getReferencedImagesList(ContentSequence),
     measurements: _getMeasurements(ContentSequence),
+    sopClassUids,
   };
 
   console.log(DisplaySetService);
@@ -386,8 +387,6 @@ function _processNonGeometricallyDefinedMeasurement(mergedContentSequence) {
     TrackingUniqueIdentifier: UIDREFContentItem.UID,
   };
 
-  debugger;
-
   NUMContentItems.forEach(item => {
     const {
       ConceptNameCodeSequence,

extensions/dicom-sr/src/index.js

@@ -34,13 +34,15 @@ export default {
    * @param {object} [configuration={}]
    * @param {object|array} [configuration.csToolsConfig] - Passed directly to `initCornerstoneTools`
    */
-  getViewportModule({ commandsManager }) {
+  getViewportModule({ servicesManager }) {
     const ExtendedOHIFCornerstoneSRViewport = props => {
-      const onNewImageHandler = jumpData => {
+      const { DisplaySetService } = servicesManager.services;
-        commandsManager.runCommand('jumpToImage', jumpData);
+
-      };
       return (
-        <OHIFCornerstoneSRViewport {...props} onNewImage={onNewImageHandler} />
+        <OHIFCornerstoneSRViewport
+          DisplaySetService={DisplaySetService}
+          {...props}
+        />
       );
     };
 

extensions/dicom-sr/src/init.js

@@ -1,5 +1,7 @@
 import cornerstoneTools from 'cornerstone-tools';
 import DICOMSRDisplayTool from './tools/DICOMSRDisplayTool';
+import dicomSRModule from './tools/modules/dicomSRModule';
+import id from './id';
 
 import TOOL_NAMES from './constants/toolNames';
 
@@ -17,5 +19,7 @@ export default function init({ configuration = {} }) {
 
   TOOL_NAMES.DICOM_SR_DISPLAY_TOOL = conifg.TOOL_NAMES.DICOM_SR_DISPLAY_TOOL;
 
+  cornerstoneTools.register('module', id, dicomSRModule);
   cornerstoneTools.addTool(DICOMSRDisplayTool);
+  cornerstoneTools.setToolEnabled(TOOL_NAMES.DICOM_SR_DISPLAY_TOOL);
 }

extensions/dicom-sr/src/tools/DICOMSRDisplayTool.js

@@ -2,6 +2,7 @@ import { importInternal, getToolState, toolColors } from 'cornerstone-tools';
 
 import TOOL_NAMES from '../constants/toolNames';
 import SCOORD_TYPES from '../constants/scoordTypes';
+import id from '../id';
 
 // Cornerstone 3rd party dev kit imports
 const draw = importInternal('drawing/draw');
@@ -25,11 +26,14 @@ export default class DICOMSRDisplayTool extends BaseTool {
     const initialProps = Object.assign(defaultProps, props);
 
     super(initialProps);
+
+    this._module = cornerstoneTools.getModule(id);
   }
 
   renderToolData(evt) {
     const eventData = evt.detail;
     const { element } = eventData;
+    const module = this._module;
 
     const toolState = getToolState(element, this.name);
 
@@ -37,36 +41,66 @@ export default class DICOMSRDisplayTool extends BaseTool {
       return;
     }
 
-    const options = {
+    const trackingUniqueIdentifiersForElement = module.getters.trackingUniqueIdentifiersForElement(
-      color: toolColors.setToolColor(),
+      element
-      lineWidth: 2,
+    );
-      handleRadius: 6,
-    };
 
-    for (let i = 0; i < toolState.data.length; i++) {
+    const {
-      const data = toolState.data[i];
+      activeIndex,
+      trackingUniqueIdentifiers,
+    } = trackingUniqueIdentifiersForElement;
+
+    const activeTrackingUniqueIdentifier =
+      trackingUniqueIdentifiers[activeIndex];
+
+    // Filter toolData to only render the data for the active SR.
+    const filteredToolData = toolState.data.filter(td =>
+      trackingUniqueIdentifiers.includes(td.TrackingUniqueIdentifier)
+    );
+
+    for (let i = 0; i < filteredToolData.length; i++) {
+      const data = filteredToolData[i];
       const { renderableData } = data;
 
+      const color =
+        data.TrackingUniqueIdentifier === activeTrackingUniqueIdentifier
+          ? toolColors.getActiveColor()
+          : toolColors.getToolColor();
+
+      const options = {
+        color,
+        lineWidth: 2,
+        handleRadius: 6,
+      };
+
       Object.keys(renderableData).forEach(GraphicType => {
         const renderableDataForGraphicType = renderableData[GraphicType];
 
         switch (GraphicType) {
           case SCOORD_TYPES.POINT:
           case SCOORD_TYPES.MULTIPOINT:
-            renderPointOrMultipoint(
+            this.renderPointOrMultipoint(
               renderableDataForGraphicType,
               eventData,
               options
             );
             break;
           case SCOORD_TYPES.POLYLINE:
-            renderPolyLine(renderableDataForGraphicType, eventData, options);
+            this.renderPolyLine(
+              renderableDataForGraphicType,
+              eventData,
+              options
+            );
             break;
           case SCOORD_TYPES.CIRCLE:
-            renderCircle(renderableDataForGraphicType, eventData, options);
+            this.renderCircle(renderableDataForGraphicType, eventData, options);
             break;
           case SCOORD_TYPES.ELLIPSE:
-            renderEllipse(renderableDataForGraphicType, eventData, options);
+            this.renderEllipse(
+              renderableDataForGraphicType,
+              eventData,
+              options
+            );
             break;
         }
       });

extensions/dicom-sr/src/tools/modules/dicomSRModule.js

@@ -0,0 +1,61 @@
+import cornerstone from 'cornerstone-core';
+
+const state = {
+  TrackingUniqueIdentifier: null,
+  trackingIdentifiersByEnabledElementUUID: {},
+};
+
+function setTrackingUniqueIdentifiersForElement(
+  element,
+  trackingUniqueIdentifiers,
+  activeIndex = 0
+) {
+  const enabledElement = cornerstone.getEnabledElement(element);
+  const { uuid } = enabledElement;
+
+  state.trackingIdentifiersByEnabledElementUUID[uuid] = {
+    trackingUniqueIdentifiers,
+    activeIndex,
+  };
+}
+
+function setActiveTrackingUniqueIdentifierForElement(
+  element,
+  TrackingUniqueIdentifier
+) {
+  const enabledElement = cornerstone.getEnabledElement(element);
+  const { uuid } = enabledElement;
+
+  const trackingIdentifiersForElement =
+    state.trackingIdentifiersByEnabledElementUUID[uuid];
+
+  if (trackingIdentifiersForElement) {
+    const activeIndex = trackingIdentifiersForElement.trackingUniqueIdentifiers.findIndex(
+      tuid => tuid === TrackingUniqueIdentifier
+    );
+
+    trackingIdentifiersForElement.activeIndex = activeIndex;
+  }
+}
+
+function getTrackingUniqueIdentifiersForElement(element) {
+  const enabledElement = cornerstone.getEnabledElement(element);
+  const { uuid } = enabledElement;
+
+  if (state.trackingIdentifiersByEnabledElementUUID[uuid]) {
+    return state.trackingIdentifiersByEnabledElementUUID[uuid];
+  }
+
+  return { trackingUniqueIdentifiers: [] };
+}
+
+export default {
+  state,
+  getters: {
+    trackingUniqueIdentifiersForElement: getTrackingUniqueIdentifiersForElement,
+  },
+  setters: {
+    trackingUniqueIdentifiersForElement: setTrackingUniqueIdentifiersForElement,
+    activeTrackingUniqueIdentifierForElement: setActiveTrackingUniqueIdentifierForElement,
+  },
+};

extensions/dicom-sr/src/utils/addMeasurement.js

@@ -6,6 +6,8 @@ import { cornerstone } from '../../../../platform/core/src';
 const globalImageIdSpecificToolStateManager =
   cornerstoneTools.globalImageIdSpecificToolStateManager;
 
+console.log(globalImageIdSpecificToolStateManager);
+
 export default function addMeasurement(
   measurement,
   imageId,
@@ -18,8 +20,6 @@ export default function addMeasurement(
 
   // TODO -> Render rotated ellipse .
 
-  debugger;
-
   const toolName = TOOL_NAMES.DICOM_SR_DISPLAY_TOOL;
 
   const measurementData = {
@@ -62,8 +62,6 @@ export default function addMeasurement(
   measurement.imageId = imageId;
   measurement.displaySetInstanceUID = displaySetInstanceUID;
   delete measurement.coords;
-
-  debugger;
 }
 
 function _getRenderableData(GraphicType, GraphicData) {