Only provide window/level interaction with pickable image.

This brings two changes to vtkInteractorStyleImage:

1. The Window/Level interaction with an image is disabled if the image is not Pickable.

2. The user can call SetCurrentImageNumber() to specify what image the WindowLevel interaction should modify.

Basic content checks passed!

Branch-at: 1802bde6
Acked-by: @kwrobot

Restored source branch imagestyle-pickability

Added 1 commit:

• 81d49118 - Allow the user to set the image to interact with.

Basic content checks passed!

Changes since last check: compare

Branch-at: 81d491187e86c24385ea7ebf207235247e1831ac
Acked-by: @kwrobot

Added 1 commit:

• a5e87177 - Allow the user to set the image to interact with.

Basic content checks passed!

Changes since last check: compare

Branch-at: a5e87177
Acked-by: @kwrobot

Do: test

Testing commands handed to buildbot.

Branch-at: a5e87177

Your merge request has been queued for testing. You may view the test results on CDash or Buildbot.

Branch-at: a5e87177

@cory-quammen @aashish-chaudhary This is an attempt to make the WindowLevel interaction more flexible when there are multiple image layers.

For reference, this is the thread on the users list where it was discussed: http://vtk.markmail.org/message/g2gslwh7hxrw6wfk

bump

@jcfr Adding you to the review list, in case Slicer uses vtkInteractorStyleImage.

Thanks.

Since within Slicer, we currently have a custom user interactor style to handle window/level. See https://github.com/Slicer/Slicer/blob/bc7ca08a76c6e23b491bd1219d6c343756c097c8/Libs/MRML/DisplayableManager/vtkSliceViewInteractorStyle.cxx#L277-L301

I don't think your patch will directly impact Slicer.

That said, you change makes sense, it provide an easy way to enable/disable windows level. And since vtkProp are pickable by default, it won't break existing application.

Thanks for the review. I have my own interaction code, too, and I don't use this class anymore :) But I still try to maintain this class for the folks out there who use it.

Dashboard was green and I have two approvals, if I count the user who said that this fixed their problem : )

Do: merge

mentioned in commit 11b3da87

Status changed to merged

Branch successfully merged and pushed!

Mentioned in commit 11b3da87