michalkielan / MatrixLibs

Small matrix library, written C++11
Mozilla Public License 2.0
1 stars 2 forks source link

inv (nxn) not working correctly #2

Closed michalkielan closed 7 years ago

JaroslawWiosna commented 7 years ago

That's because of T det(const Matrix<T, n, n>& A) (n >= 4) is not implemented.

Please, consider changing status to blocked, and creating separate issue about det [n>=4]

Once det() for n>=4 is implemented, inv() for n>=4 should be working correctly.

michalkielan commented 7 years ago

try on 3x3, or hardcode detA

michalkielan commented 7 years ago

Nie działa Panie, :P, dodaj doxygenowe komenty w concagenate

On 01.10.2016 13:43, JaroslawWiosna wrote:

That's because of |T det(const Matrix<T, n, n>& A)| (n >= 4) /is not implemented./

Please, consider changing status to blocked, and creating separate issue about det [n>=4]

Once /det() for n>=4/ is implemented, /inv() for n>=4/ should be working correctly.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/michal915/MatrixLibs/issues/2#issuecomment-250908013, or mute the thread https://github.com/notifications/unsubscribe-auth/ADuSMbt1T2Bn8soxOAkRKmrogu-fzCh0ks5qvkdNgaJpZM4KLB8R.

JaroslawWiosna commented 7 years ago

check #5

JaroslawWiosna commented 7 years ago

What was wrong:

ATTENTION! In tests I used matrix 4x4 with det=-1 ! That's because det(nxn) n>=4 is hardcoded!