Skip to content

Commit 7fbbaa8

Browse files
florystfloryst
committed
refactor(OpenGL/Texture): use named parameters
The create* methods have enough parameters to warrant using named parameters. BREAKING CHANGE: the create* methods now use named parameters rather than positional parameters.
1 parent 24a81a9 commit 7fbbaa8

File tree

12 files changed

+461
-375
lines changed

12 files changed

+461
-375
lines changed

‎Sources/Rendering/OpenGL/Framebuffer/index.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -244,13 +244,13 @@ function vtkFramebuffer(publicAPI, model) {
244244
texture.setOpenGLRenderWindow(model._openGLRenderWindow);
245245
texture.setMinificationFilter(Filter.LINEAR);
246246
texture.setMagnificationFilter(Filter.LINEAR);
247-
texture.create2DFromRaw(
248-
model.glFramebuffer.width,
249-
model.glFramebuffer.height,
250-
4,
251-
VtkDataTypes.UNSIGNED_CHAR,
252-
null
253-
);
247+
texture.create2DFromRaw({
248+
width: model.glFramebuffer.width,
249+
height: model.glFramebuffer.height,
250+
numComps: 4,
251+
dataType: VtkDataTypes.UNSIGNED_CHAR,
252+
data: null,
253+
});
254254
publicAPI.setColorBuffer(texture);
255255

256256
// for now do not count on having a depth buffer texture

‎Sources/Rendering/OpenGL/ImageCPRMapper/index.js

Lines changed: 42 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -213,13 +213,13 @@ function vtkOpenGLImageCPRMapper(publicAPI, model) {
213213
model.context.getExtension('EXT_texture_norm16')
214214
);
215215
model.volumeTexture.resetFormatAndType();
216-
model.volumeTexture.create3DFilterableFromDataArray(
217-
dims[0],
218-
dims[1],
219-
dims[2],
220-
scalars,
221-
model.renderable.getPreferSizeOverAccuracy()
222-
);
216+
model.volumeTexture.create3DFilterableFromDataArray({
217+
width: dims[0],
218+
height: dims[1],
219+
depth: dims[2],
220+
dataArray: scalars,
221+
preferSizeOverAccuracy: model.renderable.getPreferSizeOverAccuracy(),
222+
});
223223
model._openGLRenderWindow.setGraphicsResourceForObject(
224224
scalars,
225225
model.volumeTexture,
@@ -246,14 +246,13 @@ function vtkOpenGLImageCPRMapper(publicAPI, model) {
246246
property.setUpdatedExtents([]);
247247

248248
const dims = image.getDimensions();
249-
model.volumeTexture.create3DFilterableFromDataArray(
250-
dims[0],
251-
dims[1],
252-
dims[2],
253-
scalars,
254-
false,
255-
updatedExtents
256-
);
249+
model.volumeTexture.create3DFilterableFromDataArray({
250+
width: dims[0],
251+
height: dims[1],
252+
depth: dims[2],
253+
dataArray: scalars,
254+
updatedExtents,
255+
});
257256
}
258257

259258
// Rebuild the color texture if needed
@@ -309,27 +308,27 @@ function vtkOpenGLImageCPRMapper(publicAPI, model) {
309308
}
310309
}
311310
model.colorTexture.resetFormatAndType();
312-
model.colorTexture.create2DFromRaw(
313-
cWidth,
314-
textureHeight,
315-
3,
316-
VtkDataTypes.UNSIGNED_CHAR,
317-
cTable
318-
);
311+
model.colorTexture.create2DFromRaw({
312+
width: cWidth,
313+
height: textureHeight,
314+
numComps: 3,
315+
dataType: VtkDataTypes.UNSIGNED_CHAR,
316+
data: cTable,
317+
});
319318
} else {
320319
for (let i = 0; i < cWidth * 3; ++i) {
321320
cTable[i] = (255.0 * i) / ((cWidth - 1) * 3);
322321
cTable[i + 1] = (255.0 * i) / ((cWidth - 1) * 3);
323322
cTable[i + 2] = (255.0 * i) / ((cWidth - 1) * 3);
324323
}
325324
model.colorTexture.resetFormatAndType();
326-
model.colorTexture.create2DFromRaw(
327-
cWidth,
328-
1,
329-
3,
330-
VtkDataTypes.UNSIGNED_CHAR,
331-
cTable
332-
);
325+
model.colorTexture.create2DFromRaw({
326+
width: cWidth,
327+
height: 1,
328+
numComps: 3,
329+
dataType: VtkDataTypes.UNSIGNED_CHAR,
330+
data: cTable,
331+
});
333332
}
334333

335334
if (firstColorTransferFunc) {
@@ -407,24 +406,24 @@ function vtkOpenGLImageCPRMapper(publicAPI, model) {
407406
}
408407
}
409408
model.pwfTexture.resetFormatAndType();
410-
model.pwfTexture.create2DFromRaw(
411-
pwfWidth,
412-
textureHeight,
413-
1,
414-
VtkDataTypes.FLOAT,
415-
pwfFloatTable
416-
);
409+
model.pwfTexture.create2DFromRaw({
410+
width: pwfWidth,
411+
height: textureHeight,
412+
numComps: 1,
413+
dataType: VtkDataTypes.FLOAT,
414+
data: pwfFloatTable,
415+
});
417416
} else {
418417
// default is opaque
419418
pwfTable.fill(255.0);
420419
model.pwfTexture.resetFormatAndType();
421-
model.pwfTexture.create2DFromRaw(
422-
pwfWidth,
423-
1,
424-
1,
425-
VtkDataTypes.UNSIGNED_CHAR,
426-
pwfTable
427-
);
420+
model.pwfTexture.create2DFromRaw({
421+
width: pwfWidth,
422+
height: 1,
423+
numComps: 1,
424+
dataType: VtkDataTypes.UNSIGNED_CHAR,
425+
data: pwfTable,
426+
});
428427
}
429428
if (firstPwFunc) {
430429
model._openGLRenderWindow.setGraphicsResourceForObject(

‎Sources/Rendering/OpenGL/ImageMapper/index.js

Lines changed: 45 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -1001,26 +1001,26 @@ function vtkOpenGLImageMapper(publicAPI, model) {
10011001
}
10021002
}
10031003
model.colorTexture.resetFormatAndType();
1004-
model.colorTexture.create2DFromRaw(
1005-
cWidth,
1006-
textureHeight,
1007-
3,
1008-
VtkDataTypes.UNSIGNED_CHAR,
1009-
cTable
1010-
);
1004+
model.colorTexture.create2DFromRaw({
1005+
width: cWidth,
1006+
height: textureHeight,
1007+
numComps: 3,
1008+
dataType: VtkDataTypes.UNSIGNED_CHAR,
1009+
data: cTable,
1010+
});
10111011
} else {
10121012
for (let i = 0; i < cWidth * 3; ++i) {
10131013
cTable[i] = (255.0 * i) / ((cWidth - 1) * 3);
10141014
cTable[i + 1] = (255.0 * i) / ((cWidth - 1) * 3);
10151015
cTable[i + 2] = (255.0 * i) / ((cWidth - 1) * 3);
10161016
}
1017-
model.colorTexture.create2DFromRaw(
1018-
cWidth,
1019-
1,
1020-
3,
1021-
VtkDataTypes.UNSIGNED_CHAR,
1022-
cTable
1023-
);
1017+
model.colorTexture.create2DFromRaw({
1018+
width: cWidth,
1019+
height: 1,
1020+
numComps: 3,
1021+
dataType: VtkDataTypes.UNSIGNED_CHAR,
1022+
data: cTable,
1023+
});
10241024
}
10251025

10261026
if (firstColorTransferFunc) {
@@ -1109,23 +1109,23 @@ function vtkOpenGLImageMapper(publicAPI, model) {
11091109
}
11101110
}
11111111
model.pwfTexture.resetFormatAndType();
1112-
model.pwfTexture.create2DFromRaw(
1113-
pwfWidth,
1114-
textureHeight,
1115-
1,
1116-
VtkDataTypes.FLOAT,
1117-
pwfFloatTable
1118-
);
1112+
model.pwfTexture.create2DFromRaw({
1113+
width: pwfWidth,
1114+
height: textureHeight,
1115+
numComps: 1,
1116+
dataType: VtkDataTypes.FLOAT,
1117+
data: pwfFloatTable,
1118+
});
11191119
} else {
11201120
// default is opaque
11211121
pwfTable.fill(255.0);
1122-
model.pwfTexture.create2DFromRaw(
1123-
pwfWidth,
1124-
1,
1125-
1,
1126-
VtkDataTypes.UNSIGNED_CHAR,
1127-
pwfTable
1128-
);
1122+
model.pwfTexture.create2DFromRaw({
1123+
width: pwfWidth,
1124+
height: 1,
1125+
numComps: 1,
1126+
dataType: VtkDataTypes.UNSIGNED_CHAR,
1127+
data: pwfTable,
1128+
});
11291129
}
11301130

11311131
if (firstPwFunc) {
@@ -1344,15 +1344,16 @@ function vtkOpenGLImageMapper(publicAPI, model) {
13441344
// Don't share this resource as `scalars` is created in this function
13451345
// so it is impossible to share
13461346
model.openGLTexture.resetFormatAndType();
1347-
model.openGLTexture.create2DFilterableFromRaw(
1348-
dims[0],
1349-
dims[1],
1350-
numComp,
1351-
imgScalars.getDataType(),
1352-
scalars,
1353-
model.renderable.getPreferSizeOverAccuracy?.(),
1354-
ranges
1355-
);
1347+
model.openGLTexture.create2DFilterableFromRaw({
1348+
width: dims[0],
1349+
height: dims[1],
1350+
numComps: numComp,
1351+
dataType: imgScalars.getDataType(),
1352+
data: scalars,
1353+
preferSizeOverAccuracy:
1354+
!!model.renderable.getPreferSizeOverAccuracy?.(),
1355+
ranges,
1356+
});
13561357
model.openGLTexture.activate();
13571358
model.openGLTexture.sendParameters();
13581359
model.openGLTexture.deactivate();
@@ -1439,13 +1440,13 @@ function vtkOpenGLImageMapper(publicAPI, model) {
14391440
model.labelOutlineThicknessTexture.setMagnificationFilter(Filter.NEAREST);
14401441

14411442
// Create a 2D texture (acting as 1D) from the raw data
1442-
model.labelOutlineThicknessTexture.create2DFromRaw(
1443-
lWidth,
1444-
lHeight,
1445-
1,
1446-
VtkDataTypes.UNSIGNED_CHAR,
1447-
lTable
1448-
);
1443+
model.labelOutlineThicknessTexture.create2DFromRaw({
1444+
width: lWidth,
1445+
height: lHeight,
1446+
numComps: 1,
1447+
dataType: VtkDataTypes.UNSIGNED_CHAR,
1448+
data: lTable,
1449+
});
14491450

14501451
if (labelOutlineThicknessArray) {
14511452
model._openGLRenderWindow.setGraphicsResourceForObject(

‎Sources/Rendering/OpenGL/ImageResliceMapper/index.js

Lines changed: 41 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -318,12 +318,12 @@ function vtkOpenGLImageResliceMapper(publicAPI, model) {
318318
model.context.getExtension('EXT_texture_norm16')
319319
);
320320
newScalarTexture.resetFormatAndType();
321-
newScalarTexture.create3DFilterableFromDataArray(
322-
dims[0],
323-
dims[1],
324-
dims[2],
325-
scalars
326-
);
321+
newScalarTexture.create3DFilterableFromDataArray({
322+
width: dims[0],
323+
height: dims[1],
324+
depth: dims[2],
325+
dataArray: scalars,
326+
});
327327
model._openGLRenderWindow.setGraphicsResourceForObject(
328328
scalars,
329329
newScalarTexture,
@@ -340,14 +340,13 @@ function vtkOpenGLImageResliceMapper(publicAPI, model) {
340340
actorProperty.setUpdatedExtents([]);
341341

342342
const dims = imageData.getDimensions();
343-
model.scalarTextures[component].create3DFilterableFromDataArray(
344-
dims[0],
345-
dims[1],
346-
dims[2],
347-
scalars,
348-
false,
349-
updatedExtents
350-
);
343+
model.scalarTextures[component].create3DFilterableFromDataArray({
344+
width: dims[0],
345+
height: dims[1],
346+
depth: dims[2],
347+
dataArray: scalars,
348+
updatedExtents,
349+
});
351350
}
352351

353352
replaceGraphicsResource(
@@ -409,13 +408,13 @@ function vtkOpenGLImageResliceMapper(publicAPI, model) {
409408
}
410409
}
411410
newColorTexture.resetFormatAndType();
412-
newColorTexture.create2DFromRaw(
413-
cWidth,
414-
textureHeight,
415-
3,
416-
VtkDataTypes.UNSIGNED_CHAR,
417-
cTable
418-
);
411+
newColorTexture.create2DFromRaw({
412+
width: cWidth,
413+
height: textureHeight,
414+
numComps: 3,
415+
dataType: VtkDataTypes.UNSIGNED_CHAR,
416+
data: cTable,
417+
});
419418
} else {
420419
for (let column = 0; column < cWidth * 3; ++column) {
421420
const opacity = (255.0 * column) / ((cWidth - 1) * 3);
@@ -427,13 +426,13 @@ function vtkOpenGLImageResliceMapper(publicAPI, model) {
427426
}
428427
}
429428
newColorTexture.resetFormatAndType();
430-
newColorTexture.create2DFromRaw(
431-
cWidth,
432-
1,
433-
3,
434-
VtkDataTypes.UNSIGNED_CHAR,
435-
cTable
436-
);
429+
newColorTexture.create2DFromRaw({
430+
width: cWidth,
431+
height: 1,
432+
numComps: 3,
433+
dataType: VtkDataTypes.UNSIGNED_CHAR,
434+
data: cTable,
435+
});
437436
}
438437

439438
if (firstColorTransferFunc) {
@@ -506,24 +505,24 @@ function vtkOpenGLImageResliceMapper(publicAPI, model) {
506505
}
507506
}
508507
newOpacityTexture.resetFormatAndType();
509-
newOpacityTexture.create2DFromRaw(
510-
pwfWidth,
511-
textureHeight,
512-
1,
513-
VtkDataTypes.FLOAT,
514-
pwfFloatTable
515-
);
508+
newOpacityTexture.create2DFromRaw({
509+
width: pwfWidth,
510+
height: textureHeight,
511+
numComps: 1,
512+
dataType: VtkDataTypes.FLOAT,
513+
data: pwfFloatTable,
514+
});
516515
} else {
517516
// default is opaque
518517
pwfTable.fill(255.0);
519518
newOpacityTexture.resetFormatAndType();
520-
newOpacityTexture.create2DFromRaw(
521-
pwfWidth,
522-
textureHeight,
523-
1,
524-
VtkDataTypes.UNSIGNED_CHAR,
525-
pwfTable
526-
);
519+
newOpacityTexture.create2DFromRaw({
520+
width: pwfWidth,
521+
height: textureHeight,
522+
numComps: 1,
523+
dataType: VtkDataTypes.UNSIGNED_CHAR,
524+
data: pwfTable,
525+
});
527526
}
528527
if (firstPwFunc) {
529528
model._openGLRenderWindow.setGraphicsResourceForObject(

0 commit comments

Comments
 (0)