Skip to content

Tilemap: Consider enabling spartial hash by default #2663

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

Open
einarf opened this issue May 3, 2025 · 2 comments
Open

Tilemap: Consider enabling spartial hash by default #2663

einarf opened this issue May 3, 2025 · 2 comments

Comments

@einarf
Copy link
Member

einarf commented May 3, 2025

I've seen a lot of of people not enabling spatial hashing in their tilemap layers. Possibly we should just do this by default because it's what most people need. Not enabling it have devastating consequences to performance.

@Cleptomania
Copy link
Member

It's a tough question, I'm not really against it, if it seems to be the common case then I'm fine with it. I think most games are realistically going to have a mix of layers that they need it on for and off for, maybe we need to do a better job of communicating it in docs?

I suppose it's probably likely that enabling spatial hash on the layers which shouldn't have it, generally speaking, is probably less substantial of a problem than not enabling it on the layers that do need it.

@einarf
Copy link
Member Author

einarf commented May 4, 2025

Still a bit on the fence about this. We can let this brew a bit longer. It implies updating examples as well. For my personal use I'd prefer it explicit just like it is now. You might be right that it's a doc issue.

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

No branches or pull requests

2 participants