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.

Sign up for GitHub

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

Jump to bottom

Update build system #7

Open
wants to merge 79 commits into
base: master
Choose a base branch
from
Open

Update build system #7

wants to merge 79 commits into from

Conversation

astro-friedel
Copy link

Updated the code for the new CMake build system for ILLIXR.

richard-elvira and others added 30 commits April 21, 2021 16:20
@richard-elvira
V0.4: Beta version, 21 April 2021
a80b467 
- Changed OpenCV dynamic matrices to static matrices to speed up the code.

- Capability to measure running time of the system threads.

- Compatibility with OpenCV 4.0 (Requires at least OpenCV 3.0)

- Fixed minor bugs.
@richard-elvira
Readme.md updated
093173b
@richard-elvira
Update README.md
2067374
@richard-elvira
Update README.md
578500b
@jj-gomez
V1.0: 22nd December 2021
0df83dd
@jdtardos
Update README.md
851db08
@jdtardos
Update Dependencies.md
4452a3c
@hungdche
Creating the plugin
69f6787
@hungdche
create setting files
bd2b22b
@hungdche
Replace stdint-gcc.h with stdint.h
1839a90
@hungdche
Commenting examples
4e1532b
@hungdche
Test
e2b68b8
@hungdche
Create Makefile
d1e9982
@hungdche
Add ILLIXR integration to make files
fbe96e8
@hungdche
Fix make files
72a7d3f
@hungdche
Fix ++ on Boolean
6fc6091
@hungdche
Test
ccd9591
@hungdche
Adding absolute paths
5a8801d
@hungdche
Fix path
7bb0c59
@hungdche
Fix path for common/
697ec8d
@hungdche
Add dummy values to debug
0602ee8
@hungdche
Test
0800733
@hungdche
Added function similar to TrackStereo
aef1fb3
@hungdche
Use tracker to grab data
c7c8531
@hungdche
Set setting to be public
4c2d009
@hungdche
Fix IMU input
68f41e9
@hungdche
Add boost filesystem tag
e61aacd
@hungdche
Deep copy images
75d6882
@JeffreyZh4ng
Fixed drifting before initialization
593abb4
@hungdche
Removing Sophus from returnTracker
ec0f440
@astro-friedel astro-friedel self-assigned this Jul 11, 2023
@hungdche hungdche added the enhancement New feature or request label Sep 6, 2023
@hungdche
Copy link
Member

hungdche commented Oct 3, 2023

Plugin now ready to be merge. Some notes:

  • We disable pose being fed during the first 2 seconds. At this time, ORB-SLAM3 only initializes stereo, not IMU. Enabling pose being fed to imu_integrator introduces undesired behavior because there are no IMU parameters available.
  • We disable the second VIBA due to little to none improvement plus VIBA makes head pose jumps when being fed to imu integrator

Results compared to open_vins:
image
image

@hungdche hungdche mentioned this pull request Oct 3, 2023
Open
@astro-friedel astro-friedel removed their assignment Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jianxiapyh jianxiapyh Awaiting requested review from jianxiapyh

@Boyuan-Tian Boyuan-Tian Awaiting requested review from Boyuan-Tian

@qinjunj qinjunj Awaiting requested review from qinjunj

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@astro-friedel @hungdche @richard-elvira @jj-gomez @jdtardos @JeffreyZh4ng @Boyuan-Tian @Jebbly @qinjunj