search

dotnet / dotnet-api-docs

.NET API reference documentation (.NET 5+, .NET Core, .NET Framework)
https://docs.microsoft.com/dotnet/api/
Other
708 stars 1.55k forks source link

Matrix.CreateLookAt handedness #2766

Open Gavin-Williams opened 5 years ago

Gavin-Williams commented 5 years ago

Hi,

Matrix4x4.CreateLookAt doesn't indicate which handedness it represents. Since right handedness is the convention, I assume that's what it is. But the method name should indicate what the handedness is for clarity and you should give an option to create the left handed representation as well. And why not use the naming convention of every other Matrix class and call it CreateLookAtRH / CreateLookAtLH ?

As it is, I don't think I can use this method - no source, no indication of it's handedness. I have no choice but to copy code from SharpDX which does respect handedness.

mairaw commented 5 years ago

@tannergooding, @ViktorHofer can one of you help with this feedback re. the docs for https://docs.microsoft.com/en-us/dotnet/api/system.numerics.matrix4x4.createlookat and its functionality?

ghost commented 2 years ago

Tagging subscribers to this area: @dotnet/area-system-numerics See info in area-owners.md if you want to be subscribed.

Issue Details
Hi, Matrix4x4.CreateLookAt doesn't indicate which handedness it represents. Since right handedness is the convention, I assume that's what it is. But the method name should indicate what the handedness is for clarity and you should give an option to create the left handed representation as well. And why not use the naming convention of every other Matrix class and call it CreateLookAtRH / CreateLookAtLH ? As it is, I don't think I can use this method - no source, no indication of it's handedness. I have no choice but to copy code from SharpDX which does respect handedness.
Author: Gavin-Williams
Assignees: -
Labels: `waiting-on-feedback`, `doc-enhancement`, `product-feedback`, `Pri3`, `area-System.Numerics`
Milestone: Backlog