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

Does the current version work with Multi-Camera arrays? #42

Open
PushyamiKaveti opened this issue Jan 5, 2023 · 2 comments
Open

Does the current version work with Multi-Camera arrays? #42

PushyamiKaveti opened this issue Jan 5, 2023 · 2 comments

Comments

@PushyamiKaveti
Copy link

Hello Authors (@Manasi94 ),
I am interested to use SVO_PRO for performing SLAM on multi-camera data (>2). I have looked into the code and tried to make it work by extending on the mono and stereo setups and using existing code stubs. But, it is failing at the initialization stage. When I probed deeper into the code I found that the Initialization method for camera arrays (ArrayInitGeometric and ArrayInitOptimization) are nlot implemented. Particularly the addFrameBundle() methods of these init classes have commented code and some methods that dont exisit (eg, trackBundleFeatures). Is there a way for the code to work with multi-camera arrays? or is it a future release? Any info on this would be great!

Thanks,
Pushyami

@WuZihao12
Copy link

Hi, I'm also interested in working with multi-camera runs, having the same problem as above.

@Xie-HIT
Copy link

Xie-HIT commented Mar 6, 2023

Hi, I think FrameHandlerArray is not work too, and I don't know why(there are several bugs and some functions are not implemented). Fortunately, I found multi-camera configuration works in FrameHandlerMono case. And you should make some changes to let it run.

  • extend makeMono() to multi-camera case
  • extend subscribeImage(), Loop(), Callback() to multi-camera case
  • comment out a CHECK_EQ() in SparseImgAlign::run()
  • fix a bug: resize track_id_vec_ and in_ba_graph_vec_ to px_vec_.cols() in DepthFilter::initializeSeeds()

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

3 participants