search

immersive-web / webxr-hand-input

A feature repo for working on hand input support in WebXR. Feature lead: Manish Goregaokar
https://immersive-web.github.io/webxr-hand-input/
Other
105 stars 17 forks source link

Unclear algorithm step #96

Closed jmajnert closed 3 years ago

jmajnert commented 3 years ago

https://github.com/immersive-web/webxr-hand-input/blob/41f27aef0cdb494705523ac1960b444339a0127f/index.bs#L345

The sentence "Populate the pose of joint in baseSpace at the time represented by frame into pose" is not clear. The dictionary gives "to populate something", not "to populate into something". The sentence should be rebuilt to clarify what is actually required here.

jmajnert commented 3 years ago

Same here: https://github.com/immersive-web/webxr-hand-input/blob/41f27aef0cdb494705523ac1960b444339a0127f/index.bs#L398

cabanier commented 3 years ago

Populate the pose links to a detailed algorithm in the WebXR spec. Are you saying that we should add a to to the beginning of the sentence?

jmajnert commented 3 years ago

Again, I was looking at FPWD. The ED does link to the algorithm that clarifies what it means. No changes needed. Closing task