search

bertiniteam / b2

Bertini 2.0: The redevelopment of Bertini in C++.
92 stars 34 forks source link

matrix traversal almost certainly incorrect #75

Open ofloveandhate opened 7 years ago

ofloveandhate commented 7 years ago

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 commented 7 years ago

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.