This repository has been archived by the owner on May 7, 2022. It is now read-only.
Replies: 1 comment 1 reply
-
Not a bad idea. Makes sense. If you want to start a PR that would be awesome :D. |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I feel that it's a little bit strange that chunks are offset from the center. I feel that chunk
(0, 0, 0)
with a chunk size of(16, 16, 16)
should be responsible for tiles from(0, 0, 0)
to(15, 15, 15)
inclusive, no? Otherwise we have to do a little bit of extra math for the conversion and requires extra mental gymnastics. Additionally, using this scheme would allow us to do some shorthands. For chunk sizes of 16, we could more easily do something likechunk_point.x << 4 + offset.x
, which feels more natural to me (and is certainly faster) than something likechunk_point.x * CHUNK_WIDTH - (CHUNK_WIDTH / 2) + offset.x
. Thoughts?Beta Was this translation helpful? Give feedback.
All reactions