Skip to content

Commit e95d606

Browse files
Changed the configurable brush sizes unit to mm
1 parent 8659cf6 commit e95d606

File tree

1 file changed

+4
-14
lines changed

1 file changed

+4
-14
lines changed

extensions/cornerstone-dicom-seg/src/panels/SegmentationToolbox.tsx

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -113,14 +113,9 @@ function SegmentationToolbox({ servicesManager, extensionManager }) {
113113

114114
const elementEnabledHandler = evt => {
115115
const setDefaultBrushSize = () => {
116-
// Brush sizes are taken as radius, so taking half the value
117-
const defaultBrushSize = (+params.get('defaultBrushSize') || 20) / 2;
118-
const minBrushSize = (+params.get('minBrushSize') || 1) / 2;
119-
const maxBrushSize = (+params.get('maxBrushSize') || 50) / 2;
120-
121-
const defaultBrushSizeInMm = convertPixelToMM(defaultBrushSize, servicesManager);
122-
let minBrushSizeInMm = convertPixelToMM(minBrushSize, servicesManager);
123-
let maxBrushSizeInMm = convertPixelToMM(maxBrushSize, servicesManager);
116+
const defaultBrushSizeInMm = +params.get('defaultBrushSize') || 2;
117+
let minBrushSizeInMm = +params.get('minBrushSize') || 2;
118+
let maxBrushSizeInMm = +params.get('maxBrushSize') || 3;
124119
const highestPixelSpacing = getPixelToMmConversionFactor(servicesManager);
125120
const lowestBrushRadius = highestPixelSpacing / 2;
126121

@@ -134,7 +129,7 @@ function SegmentationToolbox({ servicesManager, extensionManager }) {
134129
setBrushProperties({
135130
min: +minBrushSizeInMm.toFixed(2),
136131
max: +maxBrushSizeInMm.toFixed(2),
137-
step: +lowestBrushRadius.toFixed(2),
132+
step: +((maxBrushSizeInMm - minBrushSizeInMm) / 100).toFixed(2),
138133
});
139134
toolCategories.forEach(toolCategory => {
140135
onBrushSizeChange(defaultBrushSizeInMm, toolCategory);
@@ -452,11 +447,6 @@ function _getToolNamesFromCategory(category) {
452447
return toolNames;
453448
}
454449

455-
function convertPixelToMM(value, servicesManager) {
456-
const conversionFactor = getPixelToMmConversionFactor(servicesManager);
457-
return value * conversionFactor;
458-
}
459-
460450
function getPixelToMmConversionFactor(servicesManager) {
461451
const { viewportGridService, cornerstoneViewportService } = servicesManager.services;
462452
const { activeViewportId } = viewportGridService.getState();

0 commit comments

Comments
 (0)