Silk.NET icon indicating copy to clipboard operation
Silk.NET copied to clipboard

[Math][Matrix] Support both LookAtRh and LookAtLH or, at least, add the coordinate system in comments or in readme

Open guiguibubu opened this issue 1 year ago • 3 comments

Summary of feature

Add support for both coordinate system for LookAt matrix. Like it is in Stride 3D Engine https://github.com/stride3d/stride/blob/master/sources/core/Stride.Core.Mathematics/Matrix.cs

Or add some comments on the LookAt function to explicitely write the coordinate system. Or/And add it in the readme of the project

Comments

Does this have a proposal?

Check the documentation/proposals folder. If it doesn't have one, you may need to create one if you're making massive breaking changes.

guiguibubu avatar Sep 26 '24 16:09 guiguibubu

Looks like this proposal has defined this, should be implemented with 3.0 generic maths.

Perksey avatar Oct 16 '24 22:10 Perksey

Thanks for the answer ! Is there a branch to help the dev of this proposal ? I close the issue.

guiguibubu avatar Oct 17 '24 07:10 guiguibubu

I’ll keep this open until implemented

Perksey avatar Oct 17 '24 15:10 Perksey