Skip to content

Conversation

@jsjtxietian
Copy link
Contributor

As mentioned in godotengine/godot#68881 , we can update the doc for now.

I will take a look whether it's convient to change the VIEW vector to vec3(0, 0, 1) for orthogonal cameras later.

@clayjohn
Copy link
Member

clayjohn commented Jan 3, 2024

There was a discussion about overriding VIEW at some point, but I can't remember what the conclusion was. I believe there was a PR.

Edit, nevermind, there was no PR, the discussion was only in the issue you linked

@AThousandShips AThousandShips added enhancement topic:rendering area:manual Issues and PRs related to the Manual/Tutorials section of the documentation labels Jan 3, 2024
@mhilbrunner mhilbrunner merged commit 1325a33 into godotengine:master Jan 3, 2024
@mhilbrunner
Copy link
Member

Thank you! Merged.

@jsjtxietian jsjtxietian deleted the patch-2 branch January 4, 2024 02:06
mhilbrunner added a commit that referenced this pull request Jan 25, 2024
Update spatial_shader.rst to mention godot use the same VIEW vector for both perspective and orthogonal cameras.
@mhilbrunner
Copy link
Member

Cherry-picked to 4.2.

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 enhancement topic:rendering

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants