Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated run instructions #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

e3m3
Copy link

@e3m3 e3m3 commented May 13, 2021

Closes #12.

@e3m3 e3m3 added the documentation Improvements or additions to documentation label May 13, 2021
@e3m3 e3m3 requested review from charmoniumQ and mhuzai May 13, 2021 15:43
@e3m3 e3m3 self-assigned this May 13, 2021
@@ -25,7 +25,8 @@ make [opt|dbg]

Now to run OpenVINS Standalone do the following:
```
cd build/ov_msckf
# opt => <build-type> == RelWithDebInfo, dbg => <build-type> == Debug
cd build/<build-type>/ov_msckf
./run_illixr_msckf <path_to_cam0> <path_to_cam1> <path_to_imu0> <path_to_cam0_images> <path_to_cam1_images>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A lot of people have trouble with this, so let's make it explicit:
./run_illixr_msckf <path_to_cam0_data_csv> <path_to_cam1_data_csv> <path_to_imu0_data_csv> <path_to_cam0_images> <path_to_cam1_images>

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The above literally just happened to Peter from Vikram's group. 😭

@qinjunj qinjunj self-requested a review December 15, 2022 17:35
@yingj4 yingj4 assigned hungdche and unassigned e3m3 Feb 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update instructions
4 participants