Muhammad Huzaifa
Muhammad Huzaifa
This is already done here: https://github.com/ILLIXR/ILLIXR/blob/c977cc519e2332bcca56f458d5bd2829e1c2560d/scripts/install_gtsam.sh#L83 So the real question is why did that line not work in the first place?
I know this was resolved in #314, but I'm not sure if `install_gtsam.sh` is as clean as it could be right now. 1. My first question is why doesn't the...
Just rebase on top of latest master and this is good to go!
I ran into this issue a few weeks ago as well, but didn't have time to look into it. My best guess is that something inside Monado is setting errno,...
This error is being caused by our Monado driver assuming a different ILLIXR runtime interface than the one that exists on master. @Hyjale, can you make a git patch for...
Compiling Monado using `clang-10`/`clang++-10` should fix this. Don't ask why because we don't know why either.
Thanks for all the help with testing. We'll keep this open for now so we don't forget the various issues that you pointed out. 1. Monado patch. @e3m3, we should...
Assuming it has been tested, looks good to me.
Some initial thoughts: - For datasets, we should have a config file and read from it; e.g., like that of [OpenVINS](https://github.com/rpng/open_vins/blob/master/ov_msckf/launch/pgeneva_ros_eth.launch) or that of [Kimera-VIO](https://github.com/MIT-SPARK/Kimera-VIO/blob/master/params/Euroc/LeftCameraParams.yaml) (the IMU file is in...
Thanks for looking into this! - Kimera has a BSD license, so no issues in using that code as a reference. That being said, using some kind of standard YAML...