b2 icon indicating copy to clipboard operation
b2 copied to clipboard

matrix traversal almost certainly incorrect

Open ofloveandhate opened this issue 8 years ago • 1 comments

we need to ensure that Eigen::Mat are being traversed according to storage order. the default is col-major, but i think our uses of it are row-major... so yeah.

ofloveandhate avatar Apr 27 '17 21:04 ofloveandhate

i fixed it in at least one place.

this doesn't make a huge difference right now, as other things completely overwhelm this in terms of time spent. but still nice to pay attention.

ofloveandhate avatar Sep 13 '17 23:09 ofloveandhate