std::ops::Neg for matrix types? #256
Closed
chubei-oppen
started this conversation in
Ideas
Replies: 3 comments 1 reply
-
|
Yeah that would be easy enough to add. I'm curious what you need it for? If you want to make a PR go for it, would need to add it for |
Beta Was this translation helpful? Give feedback.
1 reply
-
|
They could go in
https://github.com/bitshifter/glam-rs/blob/main/tests/mat3.rs#L287. There
are equivalent tests for mat2 and mat4.
…On Fri, 19 Nov 2021, 21:31 chubei-oppen, ***@***.***> wrote:
To calcuate the inverse of a rigid body transform:
[R t]
[0 1]
I want to write:
[R.transpose -R.transpose * t]
[0 1 ]
Of course I can write R.transpose * (-t) but that's an extra pair of
parentheses.
Can you kindly point out where the tests should be added? There are so
many macros.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#256 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABBEFFSHYGLOTMV7KVNV7LUMYDOPANCNFSM5IIY6E7Q>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Implemented in #258 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
The title says it all.
EDIT: Looks like as easy as a trivial impl. I tried one in https://github.com/chubei-oppen/glam-rs/commit/0a3804eee3c2f6719ec56919b3c275ff6d6b5043.
Beta Was this translation helpful? Give feedback.
All reactions