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

Path/PathFollow (+Z) wrong orientation #53837

Closed
razcore-rad opened this issue Oct 15, 2021 · 3 comments · Fixed by #72842
Closed

Path/PathFollow (+Z) wrong orientation #53837

razcore-rad opened this issue Oct 15, 2021 · 3 comments · Fixed by #72842

Comments

@razcore-rad
Copy link
Contributor

razcore-rad commented Oct 15, 2021

Godot version

v3.3.4.stable.official [faf3f88]

System information

Ubuntu 21.04

Issue description

The

Path
  - PathFollow

Setup orients on +Z, inverse from the convention. E.g. Vector3.FORWARD is Vector3(0, 0, -1) and look_at() orients on -Z, but Path/PathFollow orients on +Z.

See:

Steps to reproduce

  1. Create the following tree:
Path
  - PathFollow
  1. Set PathFollow -> Rotation Mode to Oriented.

Notice that +Z points forward on the curve. It should be -Z.

Screenshot from 2021-10-15 15-53-18

Minimal reproduction project

No response

@Calinou
Copy link
Member

Calinou commented Oct 15, 2021

Fixing this will break compatibility with existing projects, so a new property needs to be added in 3.x to invert the Z direction. (We don't need such a property in master.)

@kubecz3k
Copy link
Contributor

@Calinou since it's breaking compatibility maybe we should set milestone of this issue to 4.0?

@Calinou Calinou added this to the 4.0 milestone Oct 15, 2021
@lukostello
Copy link
Contributor

lukostello commented Nov 21, 2021

was just about to post this. I also find it odd that look_at Forward don't have a angle of 0 when using angle_to(). Makes me think what is currently RIGHT should be forward or angle 0 should be at the top. Probably wont happen because ages of convention. But if it were up to me I'd align those somehow. I'd probably make the trig functions follow the same convention as a clock. That way forward is 0 and the first quadrant is still positive for both sin and cos.

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

Successfully merging a pull request may close this issue.

5 participants