Skip to content

BUG: Update VTK backporting fix for OpenVR gesture recognition#7503

Merged
jcfr merged 1 commit intoSlicer:mainfrom
jcfr:update-vtk-with-openvr-backports
Dec 22, 2023
Merged

BUG: Update VTK backporting fix for OpenVR gesture recognition#7503
jcfr merged 1 commit intoSlicer:mainfrom
jcfr:update-vtk-with-openvr-backports

Conversation

@jcfr
Copy link
Member

@jcfr jcfr commented Dec 22, 2023

This includes a workaround to recognize OpenVR gesture where buttons are pressed consecutively.

Corresponding changes have been contributed to upstream VTK as MR-10771 and MR-10778

List of VTK changes:

$ git shortlog d5d688b47..08161371c2 --no-merges
Jean-Christophe Fillion-Robin (2):
      [Backport MR-10771] BUG: Add missing Elevation3DEvent to vtkCommand::EventHasData()
      [Backport MR-10778] BUG: Recognize OpenVR gesture where buttons are pressed consecutively

@jcfr jcfr changed the title BUG: Update VTK backporting OpenVR fix OpenVR gesture recognition BUG: Update VTK backporting fix for OpenVR gesture recognition Dec 22, 2023
This includes a workaround to recognize OpenVR gesture where buttons
are pressed consecutively.

Corresponding changes have been contributed to upstream VTK as MR-10771 [1]
and MR-10778 [2]

[1] https://gitlab.kitware.com/vtk/vtk/-/merge_requests/10771
[2] https://gitlab.kitware.com/vtk/vtk/-/merge_requests/10778

List of VTK changes:

$ git shortlog d5d688b47..08161371c2 --no-merges
Jean-Christophe Fillion-Robin (2):
      [Backport MR-10771] BUG: Add missing Elevation3DEvent to vtkCommand::EventHasData()
      [Backport MR-10778] BUG: Recognize OpenVR gesture where buttons are pressed consecutively
@jcfr jcfr force-pushed the update-vtk-with-openvr-backports branch from 95163b1 to 8e12844 Compare December 22, 2023 03:49
@jcfr
Copy link
Member Author

jcfr commented Dec 22, 2023

Branch has been locally built on the our three build machine without issue ✅

Since the key changes impact the vtkRenderingOpenVR module integrated into the SlicerVirtualReality extension, in the interested of having the SlicerVirtualReality extension available, I am bypassing moving forward with the integration 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant