hit-test icon indicating copy to clipboard operation
hit-test copied to clipboard

Fully specify coordinate system of hit test result

Open bialpio opened this issue 4 years ago • 3 comments

See the discussion for #86 for details.

/cc @Manishearth


Preview | Diff


Preview | Diff

bialpio avatar May 28 '20 04:05 bialpio

@toji @Manishearth @cabanier - PTAL, resurrected this PR given that we now have an interop issue related to underspecification of the pose. Also fixes #114.

bialpio avatar Feb 06 '23 23:02 bialpio

@toji @Manishearth @cabanier - PTAL, resurrected this PR given that we now have an interop issue related to underspecification of the pose. Also fixes #114.

Thanks for making that change. However, we talked that we might not want to have the object aligned with the ray if it's on a vertical surface. We should either make a difference between horizontal or vertical, or make the object aligned with the orientation of the hit plane.

cabanier avatar Feb 06 '23 23:02 cabanier

Thanks for making that change. However, we talked that we might not want to have the object aligned with the ray if it's on a vertical surface.

Hmm, my intuition would be to fully specify the pose to something in all cases, and let the apps decide if they want to orient placed objects differently. And since we IMO need to fully specify the pose, we may as well specify it the same way for vertical planes as we do for horizontal planes (also note that hit-tests are not guaranteed to all come from planes, so we should probably use "hit test result normal").

We should either make a difference between horizontal or vertical, or make the object aligned with the orientation of the hit plane.

Hit test pose should already be using hit test result's normal for the Y axis - is that what you are referring to here, or is it something else?

bialpio avatar Feb 06 '23 23:02 bialpio