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

R-VIO initialization #7

Closed
chutsu opened this issue Jul 10, 2019 · 3 comments
Closed

R-VIO initialization #7

chutsu opened this issue Jul 10, 2019 · 3 comments

Comments

@chutsu
Copy link

chutsu commented Jul 10, 2019

Was wondering if R-VIO needs the IMU to be stationary for a minimum time for initialization?

What I'm seeing after testing a couple MAV EuRoC bags is that the bags that start with the drone stationary at the beginning seems to initialize well. However, the MH_01_easy.bag dataset, for example, starts with the drone being moved up and down first (presumably to initialize a VO), and that seems to cause R-VIO to diverge significantly at the start. Could this be because the sliding window hasn't been filled yet and the up and down motion cause the intialization to fail? If so this might be worth mentioning in the README?

Tested:

Good Initialization --- V1_01_easy.bag -- [stationary start]
Good Initialization --- V1_02_medium.bag -- [stationary start]
Good Initialization --- V1_03_difficult.bag -- [stationary start]
Bad Initialization --- MH_01_easy.bag -- [dynamic start]
Bad Initialization --- MH_02_easy.bag -- [dynamic start]
Good Initialization --- MH_03_medium.bag -- [stationary start for 1s then up and down]

Screenshot_2019-07-10_10-09-39

^ screenshot of R-VIO running on MH_01_easy.bag

@chutsu
Copy link
Author

chutsu commented Jul 10, 2019

Just realized this is basically the same question as Issue #1

@huaizheng
Copy link
Member

huaizheng commented Jul 10, 2019

Thanks @chutsu for testing the code. The initialization that I implemented is very simple, if you refer to the corresponding part of code. So, here I would like to clarify that current R-VIO codebase is for validation of the idea proposed in our paper and provides basic navigation functionality. If you want a full-function solution (e.g., with a better initialization), then I would encourage you to implement your own one.

@chutsu
Copy link
Author

chutsu commented Jul 10, 2019

I would like to clarify that current R-VIO codebase is for validation of the idea proposed in our paper and provides basic navigation functionality

Thats fair enough. Thanks for sharing your code! :)

@chutsu chutsu closed this as completed Jul 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants