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

Load multiple non-overlapping maps #265

Closed
wants to merge 18 commits into from

Conversation

MartiBolet
Copy link

Pull Request to implement loading of multiple non-overlapping maps (#223). Added an interface to load a new map over a previous loaded one, without deleting the first one, with the possibility of scale, transpose and rotate this second map. We implemented in the example of "run_video_localization.cc" this functionality. To test this functionality we used the AIST dataset, making two maps, one using "aist_entrance_hall_1" and other with "aist_living_lab_1". Then we ran the example mentioned, with the two maps, using merged video of the two sets, one after another, and openvslam is capable of localize in one map, and when ends the first map, openvslam gets to "Lost" state but only for a few seconds before localizing to the other map.

If you have any suggestion or change to make, please make us know.

Furthermore, now we are exploring the option of load overlapping maps, since we found the difficulty of mapping large scenarios, we would like to map each section, and then join all them as a whole. How would you consider implementing this if its possible?

Thank you in advance,

@albertarla
Copy link

Link to download the video merged and individual .msg maps to test it.

@ymd-stella
Copy link
Contributor

ymd-stella commented Apr 3, 2022

Thanks!
I feel that creating another map_database and merging the two map_databases would reduce redundant codes.
In addition, now that sqlite3 is supported, I would like to support that as well.

@ymd-stella ymd-stella self-requested a review April 3, 2022 23:28
@ymd-stella ymd-stella linked an issue Apr 11, 2022 that may be closed by this pull request
2 tasks
@ymd-stella ymd-stella self-assigned this Aug 10, 2022
@ymd-stella
Copy link
Contributor

I will close this pull request as I have added a feature that can do much the same thing. See #380.

@ymd-stella ymd-stella closed this Aug 28, 2022
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

Successfully merging this pull request may close these issues.

Load multiple non-overlapping maps
3 participants