MPR slice w/ volume mapper clips

[DrewLazzeriKitware]

PR and Code Checklist

• semantic-release commit messages
• Run npm run reformat to have correctly formatted code

Context

fix #1872
Added a helper for slicing imagemapper with clip planes and removed vtkStyleInteractorMPRSlice.
The included example is similar to the vtkStyleInteractorMPRSlice example.

Changes

• Documentation and TypeScript definitions were updated to match those changes

[DrewLazzeriKitware]

@floryst There is an example which relies on InteractorStyleMPRSlice, VolumeOutline, which I'm having trouble replicating using sliceHelper. In particular, setUseLabelOutline() seems to have trouble with the way I've approached clipping, which you can see here:

setUseLabelOutline() seems important for that example. Here are images with it and without

Should I leave InteractorStyleMPRSlice in place for setUseLabelOutline?

[floryst]

I think that has to do with intermixed volumes. Though why the other example works, I have no idea. In any case, I would recommend not using the label outline example and instead just use one volume.

[agirault]

In particular, setUseLabelOutline() seems to have trouble with the way I've approached clipping, which you can see here:

I agree with @floryst that we don't need an example with two volumes, though the gif shows that there is a mismatch between work coordinates of those two datasets, so we should probably investigate the source. We should be able to render two aligned images with different orientations using two of those mappers with clipping.

[DrewLazzeriKitware]

I've removed the VolumeOutline example, and captured the issue here: #2169

The failed CI was a timeout. Is that likely just a transient issue unrelated to this change?

I am adding types to this, then should be ready to merge pending CI and review.

[DrewLazzeriKitware]

Ok types are added and CI passed this time, please review.

[finetjul]

The commit should be flagged as BREAKING_CHANGE because it removes existing files/classes

[DrewLazzeriKitware]

This is updated and ready for review again.

[DrewLazzeriKitware]

I've left the class checks in AbstractMapper but made them more general. I've also changed the removeClippingPlane method to use instances rather than indexes since this is already a breaking change.

I'm against changing AbstractMapper more in this PR unless it's related to the original issue of MPR though.

[DrewLazzeriKitware]

I've split the AbstractMapper changes into this smaller PR: #2186

I'll revisit this once that gets merged.

[DrewLazzeriKitware]

Since #2186 was merged, I've rebased this on master and it's ready for merge.