Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed issue where hands would not clear #967

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gregoryblood
Copy link

@gregoryblood gregoryblood commented Jun 27, 2021

There was a bug where hands would never clear an object after picking it up. This was due to clearanceCheckMask not being used in UpdateHand() as a layermask.

Weird issue since the variable was already made it was literally just not being used.

You can recreate the original error with previous code by placing a large trigger around the object, pick it up and drop it, then try to touch the object with the hand, the hand will go right through because it has not been cleared as it's still detecting the large trigger.

There was a bug where hands would never clear an object after picking it up. This was due to clearanceCheckMask not being used in UpdateHand() as a layermask.
@zite
Copy link
Collaborator

zite commented Feb 9, 2023

Despite taking over a year to get in we appreciate the PR. Must have been a merge issue. The fix will be in the next release.

@zite zite added the Fixed in next release Fix coming in next release label Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixed in next release Fix coming in next release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants