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

perf: ⚡️ Throttle UI refresh of synced WL value in vtk port (#1070)

Browse Source
This commit is contained in:
James Petts 2019-10-23 08:45:26 +01:00 committed by Erik Ziegler
parent 1a539d04c2
commit 9d08e81aa5
3 changed files with 17 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
extensions/vtk/package.json
View File

@ -48,7 +48,8 @@
},
"dependencies": {
"@babel/runtime": "^7.5.5",
"react-vtkjs-viewport": "^0.3.1"
"react-vtkjs-viewport": "^0.3.1",
"lodash.throttle": "^4.1.1"
},
"devDependencies": {
"@ohif/core": "^1.4.0",

4
extensions/vtk/src/OHIFVTKViewport.js
View File

@ -213,6 +213,10 @@ class OHIFVTKViewport extends Component {
volumeMapper.setSampleDistance(sampleDistance);
// Be generous to surpress warnings, as the logging really hurts performance.
// TODO: maybe we should auto adjust samples to 1000.
volumeMapper.setMaximumSamplesPerRay(4000);
volumeCache[displaySetInstanceUid] = volumeActor;
return volumeActor;

13
extensions/vtk/src/commandsModule.js
View File

@ -9,6 +9,7 @@ import {
import setMPRLayout from './utils/setMPRLayout.js';
import setViewportToVTK from './utils/setViewportToVTK.js';
import Constants from 'vtk.js/Sources/Rendering/Core/VolumeMapper/Constants.js';
import throttle from 'lodash.throttle';
const { BlendMode } = Constants;
@ -102,15 +103,23 @@ const actions = {
});
},
enableLevelTool: () => {
function updateVOI(apis, windowWidth, windowCenter) {
apis.forEach(api => {
api.updateVOI(windowWidth, windowCenter);
});
}
const throttledUpdateVOIs = throttle(updateVOI, 16, { trailing: true }); // ~ 60 fps
const callbacks = {
setOnLevelsChanged: ({ windowCenter, windowWidth }) => {
apis.forEach(api => {
const renderWindow = api.genericRenderWindow.getRenderWindow();
renderWindow.render();
api.updateVOI(windowWidth, windowCenter);
});
throttledUpdateVOIs(apis, windowWidth, windowCenter);
},
};