search

SouthEugeneRoboticsTeam / dunk

A fast ICP SLAM implementation for FRC robots
GNU General Public License v3.0
9 stars 1 forks source link

Fix CMakeLists.txt #9

Open andrewda opened 5 years ago

andrewda commented 5 years ago

Looks like MRPT deprecated some stuff https://travis-ci.org/SouthEugeneRoboticsTeam/dunk/builds/480756943?utm_source=github_status&utm_medium=notification

(ping @JacksonCoder help plz)

andrewda commented 5 years ago

Trying to fix this on the fix-cmake branch but failing miserably...

snowsignal commented 5 years ago

Let me look into this.

snowsignal commented 5 years ago

One sec, I need to compile MRPT manually before I can test this.

snowsignal commented 5 years ago

Alright, I've spent over 5 hours on this, and my conclusion is that MRPT is very much broken. I'm going to ask on the Google Groups that they have what I'm doing wrong. I was actually able to fix the issues I was having with the include paths, but unfortunately, now it's saying the internal include file paths cannot be found... and there are hundreds of them. There has to be another way, but neither pkg-config or CMake automatically adds these paths.

andrewda commented 5 years ago

Yikes okay. Hopefully the Google Group gets back to us quick.

On Fri, Jan 18, 2019, 7:43 AM Jackson Lewis <notifications@github.com wrote:

Alright, I've spent over 5 hours on this, and my conclusion is that MRPT is very much broken. I'm going to ask on the Google Groups that they have what I'm doing wrong. I was actually able to fix the issues I was having with the include paths, but unfortunately, now it's saying the internal include file paths cannot be found... and there are hundreds of them. There has to be another way, but neither pkg-config or CMake automatically add these paths.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/SouthEugeneRoboticsTeam/dunk/issues/9#issuecomment-455589597, or mute the thread https://github.com/notifications/unsubscribe-auth/AJuA7JDJ0zjHaj2ufrbQzw19bAcfxTFfks5vEeuKgaJpZM4aEsHP .

snowsignal commented 5 years ago

Well... the last response from them was on November 11th... so I'm not too hopeful.

In the meantime, I'll look for another SLAM library.

snowsignal commented 5 years ago

https://github.com/YoshuaNava/icpslam

I'll do some experimentation with this, see if it can potentially work.