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

Editor crashes when re-baking NavigationMesh #87267

Closed
AndresDavidCalderon opened this issue Jan 16, 2024 · 4 comments
Closed

Editor crashes when re-baking NavigationMesh #87267

AndresDavidCalderon opened this issue Jan 16, 2024 · 4 comments

Comments

@AndresDavidCalderon
Copy link

Tested versions

Reproducible in 4.2.1 stable

System information

Godot v4.2.1.stable - Windows 10.0.22621 - Vulkan (Forward+) - integrated AMD Radeon(TM) Graphics (Advanced Micro Devices, Inc.; 31.0.21905.1001) - AMD Ryzen 5 5600G with Radeon Graphics (12 Threads)

Issue description

The editor crashes when the souece geometry for a NavMesh is too big for the current cell size and cell height, and only is able to display the warning when opened using the command line.

Behaviour: Godot crashes and leaves a warning on the log giving instructions on what to do.

Expected behaviour: The editor does not crash and instead warns the user before action is taken.

Steps to reproduce

Have a project with a source geometry with an area of (in my case) 1200(x)x1(y)x2500(z) units, and leave the navigation cell size to 0.25m, with blocks of around 600(x)x7(y)x220(z) that intersect the initial block, then try to bake the navigationmesh, and the editor will crash.

Minimal reproduction project (MRP)

bignavigation.zip
Open main.tscn, try to bake the navigationMesh, and you will get an editor crash, and the warning won't display in the editor.

@smix8
Copy link
Contributor

smix8 commented Jan 16, 2024

There is little that can be done here without modifying a thirdparty library. The crash happens inside the library black box when it runs out of resources due to the excessive layout that requires at least 48.000.000 voxels with those settings. It is an old issue that gets reported every other year and has been part of ReCast since forever. It is also part of the documentation.

@Scony
Copy link
Contributor

Scony commented Feb 2, 2024

As agreed with @smix8, I'll try implementing the idea proposed here: https://chat.godotengine.org/channel/navigation?msg=ZkocD7ZRbaDgTRCoB

@Scony
Copy link
Contributor

Scony commented Mar 24, 2024

I think we can close this as #89692 landed already.

@smix8 smix8 added this to the 4.3 milestone Mar 27, 2024
@KoBeWi
Copy link
Member

KoBeWi commented Apr 4, 2024

Fixed by #89692

@KoBeWi KoBeWi closed this as completed Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants