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

Implement Heightfield Collider #4

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Rover656
Copy link

This adds Heightfield Collisions to the Terrain. This works with terrain modification too.

@Rover656
Copy link
Author

Apologies for changing the formats, tomorrow I will fix the formatting and will also work on fixing a couple glitches I am encountering.

@Rover656
Copy link
Author

Currently working on fixing this, something broke somewhere!

@Rover656
Copy link
Author

Okay, upon further inspection, everything works exception for the terrain modifications, I am unsure of a way to fix this.

@TomGroner
Copy link
Owner

hmm I'll try and take a peek at it and see if I can figure it out

@TomGroner
Copy link
Owner

Your height field is definitely working but it just seems to have the same problem I had a while back when I tried to use it. The collision does not always match up with the model quite right, so you modify terrain that is correct some times, but incorrect (or none) at other times. Without the debug mesh for heightfield, it's a bit difficult to figure out why 😢 Hopefully the pull request that adds the debug mesh is merged in soon.

@Rover656
Copy link
Author

Yeah, oh well, at least we are edging closer to this working properly!

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