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

The new TilemapLayer node cannot cooperate with the NavigationRegion2D node to bake polygons #91444

Closed
874808039 opened this issue May 2, 2024 · 4 comments · Fixed by #91464

Comments

@874808039
Copy link

Tested versions

Godot v4.3.dev6

System information

Godot v4.3.dev6 - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 4070 Ti (NVIDIA; 31.0.15.5161) - 13th Gen Intel(R) Core(TM) i7-13700KF (24 Threads)

Issue description

image
Collider in TilemapLayer cannot be baked with navigation polygons

Steps to reproduce

As a child node of NavigationRegion2D, TilemapLayer makes some collision bodies and clicks the bake button

Minimal reproduction project (MRP)

none

@874808039
Copy link
Author

Associate these two pulls:
#89179
#85856

@smix8
Copy link
Contributor

smix8 commented May 2, 2024

The node parsing is bound to be removed from the navmesh generators and moved to the nodes so they are responsible for their own data.

This includes the new TileMapLayer nodes. Those new node types were added without updating the navmesh generators that still only looks for a TileMap node.

@874808039
Copy link
Author

The node parsing is bound to be removed from the navmesh generators and moved to the nodes so they are responsible for their own data.

This includes the new TileMapLayer nodes. Those new node types were added without updating the navmesh generators that still only looks for a TileMap node.

Will scanning of TilemapLayer nodes be added in the future? In version 4.3

@smix8
Copy link
Contributor

smix8 commented May 2, 2024

I am working on a PR that moves the parsing code for each node type from the navmesh generators to the actual nodes. That includes the new TileMapLayer nodes.

Working on this was only now made possible as the NavigationServer parser and callback were added recently with PR #90876.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants