search

JakobEngel / dso

Direct Sparse Odometry
GNU General Public License v3.0
2.27k stars 906 forks source link

Dependency on SuiteSparse #101

Open vdanilo87 opened 6 years ago

vdanilo87 commented 6 years ago

Hi,

In readme and cmake, there is dependency on libsuitesparse, but it seems it's not used. I verified that by editing and removing traces from cmake. It would be nice to correct that because it's kind of misleading.

Thanks

NikolausDemmel commented 6 years ago

Thanks for the suggestion.

You might be right that this is a left-over from a previous version that used suitsparse, I'll have to double check the installation instructions, making sure we don't rely implicitly on a stuisparse dependency.