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

Bidirectional flag in NavigationLink3D is no longer taken into account by NavigationAgent3D #100769

Closed
HerrSersch opened this issue Dec 23, 2024 · 0 comments · Fixed by #100823
Closed

Comments

@HerrSersch
Copy link
Contributor

Tested versions

  • Reproducible since d51615b was merged
  • Was working before this commit, as well as in 4.4dev7 and 4.3

System information

Godot v4.4.dev (d51615b) - Linux Mint 22 (Wilma) on X11 - X11 display driver, Multi-window, 1 monitor - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 2060 SUPER (nvidia; 550.120) - AMD Ryzen 5 7600 6-Core Processor (12 threads)

Issue description

Since commit [d51615b] was merged, NavigationAgent3D does no longer take a NavigationLink3D's Bidirectional flag into account. The agent can travel freely in both directions, even if Bidirectional is disabled.

I've attached a simple MRP with an agent that will travel to the right using a NavigationLink3D with Bidirectional disabled. The link starts on the right and ends on the left, so travel shouldn't be possible.

2024-12-23.17-52-10.mp4

Steps to reproduce

  • Start the MRP. If the agent is able to move to the right despite the fact that Bidirectional is disabled, the bug exists

Minimal reproduction project (MRP)

bidirectional_navlink_bug.zip

@AThousandShips AThousandShips added this to the 4.4 milestone Dec 23, 2024
@smix8 smix8 self-assigned this Dec 23, 2024
@akien-mga akien-mga changed the title [master] Bidirectional flag in NavigationLink3D is no longer taken into account by NavigationAgent3D Bidirectional flag in NavigationLink3D is no longer taken into account by NavigationAgent3D Dec 26, 2024
@akien-mga akien-mga moved this from Unassessed to Release Blocker in 4.x Release Blockers Dec 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Release Blocker
3 participants