Fix floor clip bug #4

Open
opened 2026-04-05 17:53:27 +02:00 by MrUnknownDE · 0 comments
Owner

Originally created by @Stermere on 3/29/2026

This PR fixes a bug that would cause floor-clip to not re-engage when the user returned to an upright pose from any pose where the hip was close enough to the floor to trigger the isStanding function to return false.

The logic that was removed here served to prevent any jumps in position as floor-clip was turning back on; however, in the current version of floor-clip, the currentDisengagementOffset variable also serves to prevent jumps in position, making the former logic redundant besides the below scenario

One potential downside of this is that there will always be a hard floor when using floor-clip, which means going down stairs or other vertical activities requiring the user to descend below their initial calibration position will behave unexpectedly. I feel this is an acceptable trade-off given a typical VR user's play space. This downside can be fixed with some changes to how currentDisengagementOffset is calculated.

*Originally created by @Stermere on 3/29/2026* This PR fixes a bug that would cause floor-clip to not re-engage when the user returned to an upright pose from any pose where the hip was close enough to the floor to trigger the `isStanding` function to return false. The logic that was removed here served to prevent any jumps in position as floor-clip was turning back on; however, in the current version of floor-clip, the `currentDisengagementOffset` variable also serves to prevent jumps in position, making the former logic redundant besides the below scenario One potential downside of this is that there will always be a hard floor when using floor-clip, which means going down stairs or other vertical activities requiring the user to descend below their initial calibration position will behave unexpectedly. I feel this is an acceptable trade-off given a typical VR user's play space. This downside can be fixed with some changes to how `currentDisengagementOffset` is calculated.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github/SlimeVR-Server#4