Skip to content

Sim Reticle Jitter Fix in Depth - #464

Open
whuang37 wants to merge 2 commits into
google:mainfrom
whuang37:sim-reticle-fix
Open

Sim Reticle Jitter Fix in Depth#464
whuang37 wants to merge 2 commits into
google:mainfrom
whuang37:sim-reticle-fix

Conversation

@whuang37

Copy link
Copy Markdown
Collaborator

Fix for the sim reticle jittering infront of the hand due to the depth mesh rendering async with the reticle intersection check #461.

Only could fix it fully by disabling reticle casting on the hands and then removing the hands from the depth creation and readding after.

There's an argument to be had this isn't 1:1 with real depth sensing since we cull the hands from the sensing but the jitter was causing alot of issues with simulator + reticle apps for me. A distance check didnt really work since its dependent on framerate + user speed and itd have to be something along the lines of .4m+ to work and cause issues.

@whuang37
whuang37 requested review from dli7319 and ruofeidu July 26, 2026 22:44
@dli7319

dli7319 commented Jul 26, 2026

Copy link
Copy Markdown
Collaborator

Fyi this often happens on device as well.
Since we need this for your use case, can we have a simulator option like excludeHandsFromDepth.
You can make it true by default.

@whuang37

whuang37 commented Jul 26, 2026

Copy link
Copy Markdown
Collaborator Author

Fyi this often happens on device as well. Since we need this for your use case, can we have a simulator option like excludeHandsFromDepth. You can make it true by default.

Potentially could just change this to a hand distance check instead and cull all reticles within some distance to the hand. It wouldn't work 100% of the time but could catch both issues on device and sim. Could be breaking though for short distance reticles so idk if worth it? Just found in simulator this was a problem 100% of the time when doing any sort of strafing.

@dli7319

dli7319 commented Jul 26, 2026

Copy link
Copy Markdown
Collaborator

We used to have something like starting the raycast at 30 cm in front of the hand. Not sure if that's still there. I may have removed it by accident.

@whuang37

Copy link
Copy Markdown
Collaborator Author

We used to have something like starting the raycast at 30 cm in front of the hand. Not sure if that's still there. I may have removed it by accident.

dont think its there anymore. Is 30 cm too far away?

@dli7319

dli7319 commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

Yeah 30 cm is too far. Lets submit this for now. I'll investigate it on device when I have time.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants