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

Missing line in tutorial about navigation with different actors #7537

Closed
1998marcom opened this issue Jun 19, 2023 · 0 comments · Fixed by #7557
Closed

Missing line in tutorial about navigation with different actors #7537

1998marcom opened this issue Jun 19, 2023 · 0 comments · Fixed by #7557
Labels
area:manual Issues and PRs related to the Manual/Tutorials section of the documentation bug

Comments

@1998marcom
Copy link

Godot version: 4.0.2

Issue description: Supposedly missing line in tutorial about navigation with different actors (and typo)

URL to the documentation page: docs.godotengine.org/en/4.0/tutorials/navigation/navigation_different_actor_types.html

The page has some good code for bootstrapping one's own code, but I had issues with regions not merging until I manually activated the navigation maps with NavigationServer3D.map_set_active(map_rid, true). If the maps are inactive by default and require manual activation, and this is their intended behaviour, I would add the previous line to the tutorial.

Also, the penultimate line should be:
var path_small_agent = NavigationServer3D.map_get_path(navigation_map_small, start_pos, end_pos, true)
rather than:
var path_small_agent = NavigationServer3D.map_get_path(navigation_mesh_small_size, start_pos, end_pos, true)
I believe it is just a clear typo.

@1998marcom 1998marcom added the bug label Jun 19, 2023
@Piralein Piralein added the area:manual Issues and PRs related to the Manual/Tutorials section of the documentation label Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:manual Issues and PRs related to the Manual/Tutorials section of the documentation bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants