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

NavigationServer example #772

Open
dirkk0 opened this issue Sep 13, 2022 · 3 comments
Open

NavigationServer example #772

dirkk0 opened this issue Sep 13, 2022 · 3 comments

Comments

@dirkk0
Copy link

dirkk0 commented Sep 13, 2022

Which demo project is affected:

https://github.com/godotengine/godot-demo-projects/tree/master/3d/navmesh

Description:
The existing Navmesh demo uses the Navigation node, which will be deprecated.
I created a minimal example using the new NavigationServer with NavigationAgent here:
https://github.com/dirkk0/godot-navigationserver-example

It may need some tweaking due to overshooting, but it works.

@smix8
Copy link
Contributor

smix8 commented Oct 31, 2022

There are 2 open prs for reworking the navigation demos for 2D and 3D. The 3D with just NavigationServer usage and the 2D with basic NavigationAgent usage. None of the demos uses or is for avoidance as the current avoidance is broken so no point in showing a broken feature. #766 #765

This demo here has issues, like using the set_target_location every physics frame (resets the entire path and makes agents dance in place in bad situations) or showing signal callback for avoidance without showing the correct use of the avoidance. While I think there is place for a RigidBody or avoidance using demo I don't think it should be added as a demo like this. As mentioned I did not try to add a demo for avoidance until the avoidance starts to work properly in Godot.

@dirkk0
Copy link
Author

dirkk0 commented Oct 31, 2022

Got it.
Do you want me to rework the demo?
According to your comment, I guess I should rather close this issue?

@smix8
Copy link
Contributor

smix8 commented Nov 1, 2022

I think it would be good to have a demo with RigidBody or for all agent types cause they require slightly different NavigationAgent scripts to function properly. I think a single demo with the different agent types e.g. Node3D, CharacterBody3D and RigidBody3D would be good and the same for 2D. If the open prs could be merged the existing demos could be extended in this direction. This issue here could stay open to track this.

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

2 participants