OpenXR-Docs icon indicating copy to clipboard operation
OpenXR-Docs copied to clipboard

Clarify `jointParents` value for root bones in `XR_FB_hand_tracking_mesh`

Open flashk opened this issue 2 years ago • 4 comments

I'm adding support for the XR_FB_hand_tracking_mesh extension to my application, and it wasn't clear to me what the jointParents value of the XrHandTrackingMeshFB struct would be for root bones. Based on my testing it appears the joint parent value for root bones is XR_HAND_JOINT_MAX_ENUM_EXT. I think it would be useful to clarify that in the specification.

flashk avatar Jun 10 '23 16:06 flashk

Hmm, MAX_ENUM shouldn't actually be used anywhere, it solely exists to force the size of an enum type due to annoying limitations of C. I'll see if I can ping somebody who can answer this.

rpavlik avatar Jun 14 '23 17:06 rpavlik

An issue (number 2038) has been filed to correspond to this issue in the internal Khronos GitLab (Khronos members only: KHR:openxr/openxr#2038 ), to facilitate working group processes.

This GitHub issue will continue to be the main site of discussion.

rpavlik-bot avatar Jun 21 '23 22:06 rpavlik-bot

I'm adding support for the XR_FB_hand_tracking_mesh extension to my application, and it wasn't clear to me what the jointParents value of the XrHandTrackingMeshFB struct would be for root bones. Based on my testing it appears the joint parent value for root bones is XR_HAND_JOINT_MAX_ENUM_EXT. I think it would be useful to clarify that in the specification.

Sorry for the delay on this. I have confirmed that the this is the way that the extension is intended to work (returning XR_HAND_JOINT_MAX_ENUM_EXT for the parent of the wrist joint). I will update the documentation to reflect this. It would likely cause more problems to change it.

RobertC-Meta avatar Jul 13 '23 20:07 RobertC-Meta

Great, thanks for the clarification.

flashk avatar Jul 20 '23 13:07 flashk