Skip to content

Terrain-relative height proof of concept#44

Open
AnyOldName3 wants to merge 1 commit intopelicanmapping:mainfrom
AnyOldName3:terrain-relative-position-proof-of-concept
Open

Terrain-relative height proof of concept#44
AnyOldName3 wants to merge 1 commit intopelicanmapping:mainfrom
AnyOldName3:terrain-relative-position-proof-of-concept

Conversation

@AnyOldName3
Copy link
Copy Markdown
Contributor

This isn't really intended to be merged - I've basically taken a stab at what was discussed here #33 and would like a general idea of how horrifying the implementation is from someone who has a better idea of how things are supposed to work.

I used the class name suggested in the discussion, but it's probably not a good choice as it might get confusing with there also being an ECS that also has entities.

It should probably automatically dirty itself when new LOD is streamed in or out as otherwise a feature that is only represented at certain LOD levels may lead to a node ending up underground or floating. I've not implemented this yet, though.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

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