search

nmbr73 / Shaderfuse

Shadertoy.com WebGL shaders converted to DCTL Fuses for use with Version 18 of Blackmagic Design's DaVinci Resolve and/or BMD's Fusion running on Windows (with nvidia GPU) or (Apple Silicon) macOS.
Other
50 stars 11 forks source link

Matrix multiplication #13

Closed nmbr73 closed 3 years ago

nmbr73 commented 3 years ago

Not used to GitHub 'Issues' - Will mal ausprobieren, wie das mit den Issues geht ...

File ObjektShader/DancyTreeDoodle3D.fuse

I think there is a small bug in `float2 f2_multi_mat2(float2 A, mat2 B)' - imho it should be:

  C.x = A.x * B.r0.x + A.y * B.r1.x;  
  C.y = A.x * B.r0.y + A.y * B.r1.y;

... aber ich vermute, daß man in so einem Fall einfach die Korrektur vornimmt und sie vom anderen per 'pull request' begutachten lässt.

J-i-P-i commented 3 years ago

Habe es gereviewt - und verstehe manchmal auch nicht die veschlungenen Pfade der Matrizenmultiplikation - Ich habe einfach beide Varianten und nehme die, die funktioniert :-)