-
Notifications
You must be signed in to change notification settings - Fork 350
Hexagon #2695
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
base: development
Are you sure you want to change the base?
Hexagon #2695
Conversation
benjamin-kirkbride
commented
May 20, 2025
Let's not re-format all the test 😄 |
maybe also somewhere in the documentation reference https://www.redblobgames.com/grids/hexagons/? As its a really good primer on a lot of the hex logic that's being used here esp around the coordinate spaces |
I'm also confused why this PR seems to include a bunch of work on the platformer tutorial. Is it just a merge issue or are there two PRs accidentally stuck together? |
Sorry about the formatting stuff, I was rushing to get this done before leaving PyCon. I'll remove that stuff |
All of what I did was based on that, so this is a good idea if someone wants to pick that up and do it |
e10c264
to
9205c1d
Compare
Okay @einarf @DragonMoffon we should be good now. Sorry about the mess lol |
Reviewing this again I realized that this isn't really a hexagon shape? I still think the cube coordinate helpers are useful, but we should also make a Hexagon shape that matches with |
Yeah this is for supporting hexagon tile maps. Sorry, could have made that clear |
I think what I'm building now will support this endeavor, so no harm no foul! Glad you did this work, I think it's good for the library, I'm only worried about naming stuff. |
Maybe my object should be HexTile or something? |
If that's a compromise you're willing to make, I like that. |