diff --git a/extensions/cornerstone-dicom-rt/package.json b/extensions/cornerstone-dicom-rt/package.json index cd4bcdb6c7a..89f8071a9d7 100644 --- a/extensions/cornerstone-dicom-rt/package.json +++ b/extensions/cornerstone-dicom-rt/package.json @@ -42,20 +42,20 @@ "webpack-merge": "^5.7.3" }, "dependencies": { - "@babel/runtime": "7.7.6", + "@babel/runtime": "^7.20.13", "react-color": "^2.19.3" }, "devDependencies": { - "@babel/core": "^7.5.0", + "@babel/core": "^7.21.4", "@babel/plugin-proposal-class-properties": "^7.5.0", "@babel/plugin-proposal-object-rest-spread": "^7.5.5", "@babel/plugin-syntax-dynamic-import": "^7.2.0", "@babel/plugin-transform-arrow-functions": "^7.2.0", - "@babel/plugin-transform-regenerator": "^7.4.5", - "@babel/plugin-transform-runtime": "^7.5.0", + "@babel/plugin-transform-regenerator": "^7.16.7", + "@babel/plugin-transform-runtime": "^7.17.0", "babel-plugin-inline-react-svg": "^2.0.1", "@babel/preset-env": "^7.5.0", - "@babel/preset-react": "^7.0.0", + "@babel/preset-react": "^7.16.7", "babel-eslint": "^8.0.3", "babel-loader": "^8.0.0-beta.4", "clean-webpack-plugin": "^4.0.0", diff --git a/extensions/cornerstone-dicom-rt/src/viewports/OHIFCornerstoneRTViewport.tsx b/extensions/cornerstone-dicom-rt/src/viewports/OHIFCornerstoneRTViewport.tsx index b7708968c98..58d190f5a86 100644 --- a/extensions/cornerstone-dicom-rt/src/viewports/OHIFCornerstoneRTViewport.tsx +++ b/extensions/cornerstone-dicom-rt/src/viewports/OHIFCornerstoneRTViewport.tsx @@ -113,7 +113,6 @@ function OHIFCornerstoneRTViewport(props) { }} onElementEnabled={onElementEnabled} onElementDisabled={onElementDisabled} - // initialImageIndex={initialImageIndex} > ); }, [viewportIndex, rtDisplaySet, toolGroupId]); diff --git a/extensions/cornerstone-dicom-seg/package.json b/extensions/cornerstone-dicom-seg/package.json index d0cae55b5ef..577f9dc942e 100644 --- a/extensions/cornerstone-dicom-seg/package.json +++ b/extensions/cornerstone-dicom-seg/package.json @@ -46,16 +46,16 @@ "react-color": "^2.19.3" }, "devDependencies": { - "@babel/core": "^7.5.0", + "@babel/core": "^7.21.4", "@babel/plugin-proposal-class-properties": "^7.5.0", "@babel/plugin-proposal-object-rest-spread": "^7.5.5", "@babel/plugin-syntax-dynamic-import": "^7.2.0", "@babel/plugin-transform-arrow-functions": "^7.2.0", - "@babel/plugin-transform-regenerator": "^7.4.5", - "@babel/plugin-transform-runtime": "^7.5.0", + "@babel/plugin-transform-regenerator": "^7.16.7", + "@babel/plugin-transform-runtime": "^7.17.0", "babel-plugin-inline-react-svg": "^2.0.1", "@babel/preset-env": "^7.5.0", - "@babel/preset-react": "^7.0.0", + "@babel/preset-react": "^7.16.7", "babel-eslint": "^8.0.3", "babel-loader": "^8.0.0-beta.4", "clean-webpack-plugin": "^4.0.0", diff --git a/extensions/cornerstone-dicom-sr/src/getSopClassHandlerModule.ts b/extensions/cornerstone-dicom-sr/src/getSopClassHandlerModule.ts index 775da7e8abc..e0209447127 100644 --- a/extensions/cornerstone-dicom-sr/src/getSopClassHandlerModule.ts +++ b/extensions/cornerstone-dicom-sr/src/getSopClassHandlerModule.ts @@ -82,13 +82,6 @@ function addInstances( // gets loaded, and to navigate among them. this.instance = this.instances[this.instances.length - 1]; this.isLoaded = false; - if (this.keyImageDisplaySet) { - this.load(); - this.keyImageDisplaySet.updateInstances(); - displaySetService.setDisplaySetMetadataInvalidated( - this.keyImageDisplaySet.displaySetInstanceUID - ); - } return this; } diff --git a/extensions/cornerstone-dicom-sr/src/utils/createReferencedImageDisplaySet.ts b/extensions/cornerstone-dicom-sr/src/utils/createReferencedImageDisplaySet.ts index 03744c26d22..a1dd42d2c23 100644 --- a/extensions/cornerstone-dicom-sr/src/utils/createReferencedImageDisplaySet.ts +++ b/extensions/cornerstone-dicom-sr/src/utils/createReferencedImageDisplaySet.ts @@ -50,7 +50,7 @@ const findReferencedInstances = ( const createReferencedImageDisplaySet = (displaySetService, displaySet) => { const instances = findReferencedInstances(displaySetService, displaySet); // This will be a member function of the created image set - const updateInstances = function () { + const updateInstances = function() { this.images.splice( 0, this.images.length, @@ -78,6 +78,7 @@ const createReferencedImageDisplaySet = (displaySetService, displaySet) => { // This object is made of multiple instances from other series isCompositeStack: true, madeInClient: true, + excludeFromThumbnailBrowser: true, updateInstances, }); diff --git a/extensions/cornerstone-dicom-sr/src/viewports/OHIFCornerstoneSRViewport.tsx b/extensions/cornerstone-dicom-sr/src/viewports/OHIFCornerstoneSRViewport.tsx index 0223ffbc9c8..8d56f400d39 100644 --- a/extensions/cornerstone-dicom-sr/src/viewports/OHIFCornerstoneSRViewport.tsx +++ b/extensions/cornerstone-dicom-sr/src/viewports/OHIFCornerstoneSRViewport.tsx @@ -7,7 +7,6 @@ import { setTrackingUniqueIdentifiersForElement } from '../tools/modules/dicomSR import { Icon, Tooltip, useViewportGrid, ViewportActionBar } from '@ohif/ui'; import hydrateStructuredReport from '../utils/hydrateStructuredReport'; -import createReferencedImageDisplaySet from '../utils/createReferencedImageDisplaySet'; const { formatDate } = utils; @@ -22,8 +21,8 @@ function OHIFCornerstoneSRViewport(props) { dataSource, displaySets, viewportIndex, - viewportOptions, viewportLabel, + viewportOptions, servicesManager, extensionManager, } = props; @@ -80,9 +79,9 @@ function OHIFCornerstoneSRViewport(props) { { servicesManager, extensionManager }, displaySetInstanceUID ); - const displaySets = srDisplaySet.keyImageDisplaySet - ? [srDisplaySet.keyImageDisplaySet] - : displaySetService.getDisplaySetsForSeries(SeriesInstanceUIDs[0]); + const displaySets = displaySetService.getDisplaySetsForSeries( + SeriesInstanceUIDs[0] + ); if (displaySets.length) { viewportGridService.setDisplaySetsForViewports([ { @@ -210,9 +209,19 @@ function OHIFCornerstoneSRViewport(props) { // should be passed second since we don't want SR displaySet to // override the activeImageDisplaySetData displaySets={[activeImageDisplaySetData]} + // It is possible that there is a hanging protocol applying viewportOptions + // for the SR, so inherit the viewport options + // TODO: Ensure the viewport options are set correctly with respect to + // stack etc, in the incoming viewport options. viewportOptions={{ ...viewportOptions, toolGroupId: `${SR_TOOLGROUP_BASE_NAME}`, + // viewportType should not be required, as the stack type should be + // required already in order to view SR, but sometimes segmentation + // views set the viewport type without fixing the allowed display + viewportType: 'stack', + // The positionIds for the viewport aren't meaningful for the child display sets + positionIds: null, }} onElementEnabled={onElementEnabled} initialImageIndex={initialImageIndex} @@ -420,18 +429,14 @@ async function _getViewportReferencedDisplaySetData( measurementSelected, displaySetService ) { - if (!displaySet.keyImageDisplaySet) { - // Create a new display set, and preserve a reference to it here, - // so that it can be re-displayed and shown inside the SR viewport. - // This is only for ease of redisplay - the display set is stored in the - // usual manner in the display set service. - displaySet.keyImageDisplaySet = createReferencedImageDisplaySet( - displaySetService, - displaySet - ); - } + const { measurements } = displaySet; + const measurement = measurements[measurementSelected]; + + const { displaySetInstanceUID } = measurement; - const referencedDisplaySet = displaySet.keyImageDisplaySet; + const referencedDisplaySet = displaySetService.getDisplaySetByUID( + displaySetInstanceUID + ); const image0 = referencedDisplaySet.images[0]; const referencedDisplaySetMetadata = { diff --git a/extensions/cornerstone/src/Viewport/OHIFCornerstoneViewport.tsx b/extensions/cornerstone/src/Viewport/OHIFCornerstoneViewport.tsx index a55d74edde2..17a4f2a0063 100644 --- a/extensions/cornerstone/src/Viewport/OHIFCornerstoneViewport.tsx +++ b/extensions/cornerstone/src/Viewport/OHIFCornerstoneViewport.tsx @@ -567,8 +567,9 @@ function _subscribeToJumpToMeasurementEvents( { referencedImageId: measurement.referencedImageId } ); } - if (cacheJumpToMeasurementEvent.cornerstoneViewport !== viewportIndex) + if (cacheJumpToMeasurementEvent.cornerstoneViewport !== viewportIndex) { return; + } _jumpToMeasurement( measurement, elementRef, diff --git a/extensions/cornerstone/src/services/SegmentationService/SegmentationService.ts b/extensions/cornerstone/src/services/SegmentationService/SegmentationService.ts index 5e5642727c9..26d47b3f731 100644 --- a/extensions/cornerstone/src/services/SegmentationService/SegmentationService.ts +++ b/extensions/cornerstone/src/services/SegmentationService/SegmentationService.ts @@ -1247,37 +1247,39 @@ class SegmentationService extends PubSubService { const { fillAlpha } = this.getConfiguration(toolGroupId); - let count = 0; - const intervalTime = 16; - const numberOfFrames = Math.ceil(animationLength / intervalTime); + let startTime: number = null; + const animation = (timestamp: number) => { + if (startTime === null) { + startTime = timestamp; + } + + const elapsed = timestamp - startTime; + const progress = Math.min(elapsed / animationLength, 1); - this.highlightIntervalId = setInterval(() => { - const x = (count * intervalTime) / animationLength; cstSegmentation.config.setSegmentSpecificConfig( toolGroupId, segmentationRepresentation.segmentationRepresentationUID, { [segmentIndex]: { LABELMAP: { - fillAlpha: easeInOutBell(x, fillAlpha), + fillAlpha: easeInOutBell(progress, fillAlpha), }, }, } ); - count++; - - if (count === numberOfFrames) { - clearInterval(this.highlightIntervalId); + if (progress < 1) { + requestAnimationFrame(animation); + } else { cstSegmentation.config.setSegmentSpecificConfig( toolGroupId, segmentationRepresentation.segmentationRepresentationUID, {} ); - - this.highlightIntervalId = null; } - }, intervalTime); + }; + + requestAnimationFrame(animation); } private _highlightContour( diff --git a/extensions/default/src/Panels/PanelStudyBrowser.tsx b/extensions/default/src/Panels/PanelStudyBrowser.tsx index 63a614f7946..5ad698850f2 100644 --- a/extensions/default/src/Panels/PanelStudyBrowser.tsx +++ b/extensions/default/src/Panels/PanelStudyBrowser.tsx @@ -296,34 +296,36 @@ function _mapDisplaySets(displaySets, thumbnailImageSrcMap) { const thumbnailDisplaySets = []; const thumbnailNoImageDisplaySets = []; - displaySets.forEach(ds => { - const imageSrc = thumbnailImageSrcMap[ds.displaySetInstanceUID]; - const componentType = _getComponentType(ds.Modality); - - const array = - componentType === 'thumbnail' - ? thumbnailDisplaySets - : thumbnailNoImageDisplaySets; - - array.push({ - displaySetInstanceUID: ds.displaySetInstanceUID, - description: ds.SeriesDescription || '', - seriesNumber: ds.SeriesNumber, - modality: ds.Modality, - seriesDate: ds.SeriesDate, - seriesTime: ds.SeriesTime, - numInstances: ds.numImageFrames, - countIcon: ds.countIcon, - StudyInstanceUID: ds.StudyInstanceUID, - componentType, - imageSrc, - dragData: { - type: 'displayset', + displaySets + .filter(ds => !ds.excludeFromThumbnailBrowser) + .forEach(ds => { + const imageSrc = thumbnailImageSrcMap[ds.displaySetInstanceUID]; + const componentType = _getComponentType(ds.Modality); + + const array = + componentType === 'thumbnail' + ? thumbnailDisplaySets + : thumbnailNoImageDisplaySets; + + array.push({ displaySetInstanceUID: ds.displaySetInstanceUID, - // .. Any other data to pass - }, + description: ds.SeriesDescription || '', + seriesNumber: ds.SeriesNumber, + modality: ds.Modality, + seriesDate: ds.SeriesDate, + seriesTime: ds.SeriesTime, + numInstances: ds.numImageFrames, + countIcon: ds.countIcon, + StudyInstanceUID: ds.StudyInstanceUID, + componentType, + imageSrc, + dragData: { + type: 'displayset', + displaySetInstanceUID: ds.displaySetInstanceUID, + // .. Any other data to pass + }, + }); }); - }); return [...thumbnailDisplaySets, ...thumbnailNoImageDisplaySets]; } diff --git a/extensions/dicom-microscopy/package.json b/extensions/dicom-microscopy/package.json index d08fd1523b6..d3b3fc02c7b 100644 --- a/extensions/dicom-microscopy/package.json +++ b/extensions/dicom-microscopy/package.json @@ -43,15 +43,15 @@ }, "dependencies": { "@babel/runtime": "^7.20.13", - "@cornerstonejs/codec-charls": "^0.1.1", - "@cornerstonejs/codec-libjpeg-turbo-8bit": "^0.0.7", - "@cornerstonejs/codec-openjpeg": "^0.1.1", + "@cornerstonejs/codec-charls": "^1.2.3", + "@cornerstonejs/codec-libjpeg-turbo-8bit": "^1.2.2", + "@cornerstonejs/codec-openjpeg": "^1.2.2", "colormap": "^2.3", "dicom-microscopy-viewer": "^0.44.0", "dicomicc": "^0.1" }, "devDependencies": { - "@babel/core": "^7.17.8", + "@babel/core": "^7.21.4", "@babel/plugin-proposal-class-properties": "^7.16.7", "@babel/plugin-proposal-object-rest-spread": "^7.17.3", "@babel/plugin-proposal-private-methods": "^7.18.6", diff --git a/extensions/measurement-tracking/src/contexts/TrackedMeasurementsContext/TrackedMeasurementsContext.tsx b/extensions/measurement-tracking/src/contexts/TrackedMeasurementsContext/TrackedMeasurementsContext.tsx index 2a5b0a72d50..ccb95cb2e9a 100644 --- a/extensions/measurement-tracking/src/contexts/TrackedMeasurementsContext/TrackedMeasurementsContext.tsx +++ b/extensions/measurement-tracking/src/contexts/TrackedMeasurementsContext/TrackedMeasurementsContext.tsx @@ -31,12 +31,11 @@ function TrackedMeasurementsContextProvider( ) { const [viewportGrid, viewportGridService] = useViewportGrid(); const { activeViewportIndex, viewports } = viewportGrid; + const { measurementService, displaySetService } = servicesManager.services; const machineOptions = Object.assign({}, defaultOptions); machineOptions.actions = Object.assign({}, machineOptions.actions, { jumpToFirstMeasurementInActiveViewport: (ctx, evt) => { - const { measurementService } = servicesManager.services; - const { trackedStudy, trackedSeries } = ctx; const measurements = measurementService.getMeasurements(); const trackedMeasurements = measurements.filter( @@ -45,21 +44,51 @@ function TrackedMeasurementsContextProvider( trackedSeries.includes(m.referenceSeriesUID) ); - const uid = trackedMeasurements[0].uid; - console.log( 'jumping to measurement reset viewport', viewportGrid.activeViewportIndex, trackedMeasurements[0] ); + + const referencedDisplaySetUID = + trackedMeasurements[0].displaySetInstanceUID; + const referencedDisplaySet = displaySetService.getDisplaySetByUID( + referencedDisplaySetUID + ); + + const referencedImages = referencedDisplaySet.images; + const isVolumeIdReferenced = referencedImages[0].imageId.startsWith( + 'volumeId' + ); + + const measurementData = trackedMeasurements[0].data; + + let imageIndex = 0; + if (!isVolumeIdReferenced && measurementData) { + // if it is imageId referenced find the index of the imageId, we don't have + // support for volumeId referenced images yet + imageIndex = referencedImages.findIndex(image => { + const imageIdToUse = Object.keys(measurementData)[0].substring(8); + return image.imageId === imageIdToUse; + }); + + if (imageIndex === -1) { + console.warn( + 'Could not find image index for tracked measurement, using 0' + ); + imageIndex = 0; + } + } + viewportGridService.setDisplaySetsForViewport({ viewportIndex: viewportGrid.activeViewportIndex, - displaySetInstanceUIDs: [trackedMeasurements[0].displaySetInstanceUID], + displaySetInstanceUIDs: [referencedDisplaySetUID], + viewportOptions: { + initialImageOptions: { + index: imageIndex, + }, + }, }); - measurementService.jumpToMeasurement( - viewportGrid.activeViewportIndex, - uid - ); }, showStructuredReportDisplaySetInActiveViewport: (ctx, evt) => { if (evt.data.createdDisplaySetInstanceUIDs.length > 0) { @@ -73,7 +102,6 @@ function TrackedMeasurementsContextProvider( } }, discardPreviouslyTrackedMeasurements: (ctx, evt) => { - const { measurementService } = servicesManager.services; const measurements = measurementService.getMeasurements(); const filteredMeasurements = measurements.filter(ms => ctx.prevTrackedSeries.includes(ms.referenceSeriesUID) @@ -85,7 +113,6 @@ function TrackedMeasurementsContextProvider( } }, clearAllMeasurements: (ctx, evt) => { - const { measurementService } = servicesManager.services; const measurements = measurementService.getMeasurements(); const measurementIds = measurements.map(fm => fm.uid); diff --git a/extensions/measurement-tracking/src/panels/PanelStudyBrowserTracking/PanelStudyBrowserTracking.tsx b/extensions/measurement-tracking/src/panels/PanelStudyBrowserTracking/PanelStudyBrowserTracking.tsx index e45dbaa6467..9cc9ab2a78c 100644 --- a/extensions/measurement-tracking/src/panels/PanelStudyBrowserTracking/PanelStudyBrowserTracking.tsx +++ b/extensions/measurement-tracking/src/panels/PanelStudyBrowserTracking/PanelStudyBrowserTracking.tsx @@ -439,124 +439,130 @@ function _mapDisplaySets( ) { const thumbnailDisplaySets = []; const thumbnailNoImageDisplaySets = []; - displaySets.forEach(ds => { - const imageSrc = thumbnailImageSrcMap[ds.displaySetInstanceUID]; - const componentType = _getComponentType(ds.Modality); - const numPanes = viewportGridService.getNumViewportPanes(); - const viewportIdentificator = - numPanes === 1 - ? [] - : viewports.reduce((acc, viewportData, index) => { - if ( - index < numPanes && - viewportData?.displaySetInstanceUIDs?.includes( - ds.displaySetInstanceUID - ) - ) { - acc.push(viewportData.viewportLabel); - } - return acc; - }, []); - - const array = - componentType === 'thumbnailTracked' - ? thumbnailDisplaySets - : thumbnailNoImageDisplaySets; - - const { displaySetInstanceUID } = ds; - - const thumbnailProps = { - displaySetInstanceUID, - description: ds.SeriesDescription, - seriesNumber: ds.SeriesNumber, - modality: ds.Modality, - seriesDate: formatDate(ds.SeriesDate), - numInstances: ds.numImageFrames, - countIcon: ds.countIcon, - StudyInstanceUID: ds.StudyInstanceUID, - componentType, - imageSrc, - dragData: { - type: 'displayset', + displaySets + .filter(ds => !ds.excludeFromThumbnailBrowser) + .forEach(ds => { + const imageSrc = thumbnailImageSrcMap[ds.displaySetInstanceUID]; + const componentType = _getComponentType(ds.Modality); + const numPanes = viewportGridService.getNumViewportPanes(); + const viewportIdentificator = + numPanes === 1 + ? [] + : viewports.reduce((acc, viewportData, index) => { + if ( + index < numPanes && + viewportData?.displaySetInstanceUIDs?.includes( + ds.displaySetInstanceUID + ) + ) { + acc.push(viewportData.viewportLabel); + } + return acc; + }, []); + + const array = + componentType === 'thumbnailTracked' + ? thumbnailDisplaySets + : thumbnailNoImageDisplaySets; + + const { displaySetInstanceUID } = ds; + + const thumbnailProps = { displaySetInstanceUID, - // .. Any other data to pass - }, - isTracked: trackedSeriesInstanceUIDs.includes(ds.SeriesInstanceUID), - viewportIdentificator, - }; - - if (componentType === 'thumbnailNoImage') { - if (dataSource.reject && dataSource.reject.series) { - thumbnailProps.canReject = true; - thumbnailProps.onReject = () => { - uiDialogService.create({ - id: 'ds-reject-sr', - centralize: true, - isDraggable: false, - showOverlay: true, - content: Dialog, - contentProps: { - title: 'Delete Report', - body: () => ( -
Are you sure you want to delete this report?
-This action cannot be undone.
-Are you sure you want to delete this report?
+This action cannot be undone.
+