Skip to content

Latest commit

 

History

History
227 lines (161 loc) · 9.6 KB

BREAKING_CHANGES.md

File metadata and controls

227 lines (161 loc) · 9.6 KB

From 24.x to 25

  • math: For functions calling math functions for computations with matrices, the format must now be number[] or Matrix as defined in the typescript definitions.
  • All widgets handles inheriting from vtkHandleRepresentation now scale up automatically.

From 23.x to 24

  • All old-style widgets except OrientationMarkerWidget and PiecewiseGaussianWidget have been removed.
Old-style/deprecated widget New-style widget
Sources/Interaction/Widgets/... Sources/Widgets/Widgets3D/...
DistanceWidget DistanceWidget
HandleWidget PolyLineWidget
ImageCroppingRegionsWidget ImageCroppingWidget
LabelWidget LabelWidget
LineWidget LineWidget
OrientationMarkerWidget (kept) not implemented
PiecewiseGaussianWidget (kept) not implemented
ResliceCursor ResliceCursorWidget
  • In SVGLandmarkRepresentation: model.showCircle is replaced by model.circleProps.visible
  • In vtk.js subclasses, prefix with '_' the following "protected" model variables:
    • vtk*: model.openglRenderWindow -> model._openglRenderWindow
    • vtk*: model.openglRenderer -> model._openglRenderer
    • vtkInteractorObserver, vtkOrientationMarkerWidget : model.interactor -> model._interactor
    • vtkAbstractWidget, vtkViewNode: model.parent -> model._parent
    • vtkProp: model.parentProp -> model._parentProp
    • vtkRenderWindowInteractor: model.view -> model._view
    • vtkRenderer: model.renderWindow -> model._renderWindow
    • vtkHardwareSelector: model.renderer -> model._renderer
    • vtkAbstractWidget: model.widgetManager -> model._widgetManager

From 22.x to 23

  • imagemapper: The original behavior of the image mapper was that if a lookup table is provided, it mapped the lookup table's scalar range by default. The new behavior disables using the lookup table scalar range by default. Instead, the window/level values are used.

From 21.x to 22

  • config/rules-linter.js is now gone.
  • AbstractMapper: Changed removeClippingPlane to use instance instead of index.

From 20.x to 21

SplineWidget's handles now scale up automatically.

From 19.x to 20

In ShapeWidget:

  • setLabelTextCallback is replaced by text substate.
  • setPixelScale has been removed. It should be replaced by point handle scale1 mixin and scaleInPixels.
  • useHandles has been removed. It should be replaced by setHandleVisibility.
  • resetAfterPointPlacement is now false by default.

RectangleWidget and EllipseWidget handles now scale up automatically.

From 19.2 to 19.3

Node >=12 is required

From 18.x to 19

vtkWidgetRepresentation.updateActorVisibility(...) lost the widgetVisibility parameter. Overrides should no longer define that parameter.

From 17.x to 18

Explicit import needs to be performed to enable rendering implementation. Profiles have been created to simplify that task. See vtk.js/Sources/Rendering/OpenGL|WebGPU/Profiles/*.

From 16.x to 17

vtkBoundingBox has been redesigned to be more light weight with static methods. A simple class has been created for backward compatibility.

From 15.x to 16

Deprecation of kw-web-suite and using a newer version of webpack for building vtk.js.

From 14.x to 15

Semantic release messed up?

From 13.x to 14

css-loader update means we need to change how css modules are configured.

From 12.x to 13

manipulators: No breaking changes internally. But the CompositeMouseManipulator no longer receives key down and key up events. I don't know of anything that this could break, but it might.

From 11.x to 12

Manipulators: manipulator lists are now actually modified by removing the manipulator from the list of manipulators, and not just replacing them by null value for a certain index. This affects the resulting size of the manipulator list, and users should not rely on an index that needed to remain constant. Getters were also removed.

From 10.x to 11

vtkPlane: return result is changed if the line intersects the plane but outside of the provided points. Previously would return intersect:false, but now intersect:true and betweenPoints:false

From 9.x to 10

PiecewiseGaussianWidget: The optional arguments of setDataArray have been gathered inside a single options object since we added to that list component and numberOfComponents

From 8.x to 9

MatrixBuilder: Remove getVTKMatrix function as it had a side-effect, was poorly named, and was not referenced anywhere else. If the user wants to transpose the matrix, they can do it after calling getMatrix()

From 7.x to 8

ToolChain: Webpack rules may differ with babel 7

From 6.x to 7

RenderWindow: OpenGL/RenderWindow::captureNextImage returns a promise with the imageURL instead of returing it directly. Core/RenderWindow::captureImages returns an array of promise.

From 5.x to 6

This switch focuses on simplifying the interactor observers, and forwarding events data to the callbacks:

  • Remove set/getAnimationState and start/stopState in InteractorStyle. Instead, use start/stop${stateName}.
  • Remove CharEvent which was a duplicate of KeyPressEvent.
  • Rename setEnable*( to setEnabled() in AbstractWidget.
  • Remove get2DPointerPosition() from AbstractWidget: the position will now be properly positioned based on the canvas bounds as soon as it is caught by the Interactor.
  • Rename Pinch events to MouseWheel events when those events are triggered by the mouse wheel, and use the wheel delta instead of a scale.
  • The interaction state is now passed through the callbacks to the events instead of being stored within the RenderWindowInteractor. That means the following API is replaced as such:
    • i.getEventPosition -> callData.position (no need for an index)
    • i.getScale -> callData.scale if pinching, or use callData.wheelDelta if you are in a MouseWheel event instead
    • i.getTranslation -> callData.translation
    • i.getRotation -> callData.rotation
    • for multitouch, touch positions are forwarded in callData.positions as {touchid1: {x:x1, y:y1, z:z1}, touchid2: {x:x2, y:y2, z:z2}}
    • every "previous" state (lastEventPosition, lastScale...) needs to be stored from the observer, for example initialized in a handleStart${event} callback and updated in the handle${event} callback
    • every callback data also holds type which is the event name
  • Remove findPokedRenderer from the InteractorStyle/Observer. Instead, the latest poked renderer is passed through the callback as callData.pokedRenderer. That renderer is never null within callbacks since the event won't forward if it is. You can also query the renderer through RenderWindowInteractor.getCurrentRenderer() but this is not recommended.
  • Add renderer as a parameter to computeWorldToDisplay and computeDisplayToWorld in InteractorObserver

Add more consistency in Readers

  • Readers should have the following set of API
    • setURL(url, options)
    • fetchData(options) // OPTIONAL and should not be required
    • parseAsArrayBuffer(ArrayBuffer) or parseAsText(String)
  • Affected readers:
    • STLReader
    • Legacy/PolyDataReader
    • ElevationReader
    • MTLReader
    • OBJReader
    • PDBReader
    • XMLReader

Refactor InteractorStyleManipulator with its manipulator

  • Rename all the classes to make them consistent
  • Update API to distinguish mode vs state (mode[drag/scroll], state[button,alt,control,shift])
  • Update impacted code and example

From 4.x to 5

Improve and normalize methods inside DataAccessHelper to allow network progress monitoring:

  • Refactor fetchText(instance = {}, url, compression, progressCallback) into fetchText(instance = {}, url, options = { compression, progressCallback: fn() })
  • Refactor fetchJSON(instance = {}, url, compression) into fetchJSON(instance = {}, baseURL, array, options = { compression, progressCallback: fn() })
  • Refactor fetchArray(instance = {}, baseURL, array, fetchGzip = false) into fetchArray(instance = {}, baseURL, array, options = { compression: 'gz', progressCallback: fn() })
  • Rename HttpDataSetReader.fetchZipFile(url) to HttpDataSetReader.fetchBinary(url, options)

From 3.x to 4

The representation module as been removed and a better approach has been taken for OBJ/MTL loading.

From 2.x to 3

Migrate to Webpack 2+ which changed the way rules are defined and how to intgrate vtk.js into your project. The following documentation has been updated accordingly.

From 1.x to 2

vtkDataSetAttributes

vtkDataSetAttributes has been reimplemented to follow the VTK model. By doing so the data model structure as changed and any code that was using it as input will need to be updated.

Before:

{ "ArrayName": { vtkDataArray instance ... } }

After:

{ "activeTCoords": -1, "activeScalars": -1, "vtkClass": "vtkDataSetAttributes", "arrays": [ { "data": { vtkDataArray instance ... } } ], "activeNormals": -1, "activeGlobalIds": -1, "activeTensors": -1, "activePedigreeIds": -1, "activeVectors": -1 }

vtkPoints

points used to be simple vtkDataArray. Now we have a vtkPoint class that contains a vtkDataArray in its data.

Before:

const coordsAsTypedArray = dataset.getPoints().getData();

After:

const dataArray = dataset.getPoints().getData(); const coordsAsTypedArray = dataArray.getData();

vtk()

Serialization model changed from { type: 'vtkPolyData', ... } to { vtkClass: 'vtkPolyData', ... }.