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

Add functionalities to fix invalid cells and clear all cells in GridMap #101902

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tygyh
Copy link
Contributor

@tygyh tygyh commented Jan 22, 2025

Adopts PR #37136 which closes #37124.

All I've done is rebase the branch.

@tygyh tygyh requested a review from a team as a code owner January 22, 2025 08:26
@tygyh tygyh changed the title Adopt Added functionalities to fix invalid cells and clear all cells in GridMap Added functionalities to fix invalid cells and clear all cells in GridMap (Adopted) Jan 22, 2025
@AThousandShips AThousandShips changed the title Added functionalities to fix invalid cells and clear all cells in GridMap (Adopted) Add functionalities to fix invalid cells and clear all cells in GridMap Jan 22, 2025
@AThousandShips AThousandShips added this to the 4.x milestone Jan 22, 2025
@tygyh tygyh force-pushed the gridmap-fix branch 2 times, most recently from 9b2411a to eb9f1f8 Compare February 14, 2025 21:19
@tygyh
Copy link
Contributor Author

tygyh commented Mar 6, 2025

I've indented and unindented the failing rows back and forth and the test still fails. Why?

@akien-mga
Copy link
Member

You're using spaces for indentation instead of tabs.

…dMap

Changes made:
* Added condition to `GridMap::set_mesh_library` to update only if new library is different from old one
* Added `GridMap::fix_invalid_cells` taking inspiration from `TileMap::fix_invalide_tiles`
* Added menu entry to clear cells in `GridMapEditor`
* Added menu entry to fix tiles in `GridMapEditor`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GridMap Mesh Library "Clear" doesn't actually clear the grid-map
4 participants