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

Added player height control to support mounting climbed objects and crouching #146

Merged
merged 1 commit into from
Jun 17, 2022

Conversation

Malcolmnixon
Copy link
Collaborator

This pull request implements feature requests #143 and #116 with the help of @teddybear082. The changes include:

  • Added advanced player height control
  • Modified climbing to collapse player to a sphere to allow mounting climbed objects
  • Added crouch movement provider

The PlayerBody now supports:

  • Player height offset to allow future height calibration (for seated or disabled players)
  • Minimum and maximum allowed player height (as measured by head position)
  • Overriding player height by movement providers (for crouching or climb-mounting)

The math for placing the player body under the camera had to be modified extensively - the original logic would always put the player body on the ARVROrigin floor, whereas the new logic puts the player body under/around the head. This change results in the head/camera always being in the correct position in relationship to the body.

Copy link
Member

@BastiaanOlij BastiaanOlij left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a really nice enhancement :)

…ayer to a sphere to allow mounting climbed objects Added crouch movement provider

Added advanced player height control Modified climbing to collapse player to a sphere to allow mounting climbed objects Added crouch movement provider
Modified height override to use 'self' as per recommendation.
@Malcolmnixon
Copy link
Collaborator Author

It should be ready to go in now.

@BastiaanOlij BastiaanOlij merged commit 5f75501 into GodotVR:master Jun 17, 2022
@BastiaanOlij
Copy link
Member

Thanks!

@Malcolmnixon Malcolmnixon deleted the climbing-mount-fix branch June 17, 2022 01:36
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.

2 participants