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

Vision Update to Main #187

Merged
merged 150 commits into from
Feb 27, 2024
Merged

Vision Update to Main #187

merged 150 commits into from
Feb 27, 2024

Conversation

a-krawciw
Copy link
Contributor

Nearly complete work-in-progress to bring the vision pipeline back online.

sherrychen127 and others added 30 commits November 23, 2022 14:25
Keeping Vision Up to Date with Main
@a-krawciw a-krawciw linked an issue Feb 19, 2024 that may be closed by this pull request
@a-krawciw a-krawciw marked this pull request as ready for review February 23, 2024 16:07
@a-krawciw
Copy link
Contributor Author

Merging of real Visual Teach and Repeat back into main!

There are a few changes to the compilation workflow to support multiple pipelines.

We have decided to make sensor selection a compile-time decision. While it is possible to decide at run-time, this ensures that the code is optimized for the sensor in use, and prevents accidental subscription to multiple sensors at the same time.

Additionally, this reduces compilation time significantly because unused pipelines are skipped.

  1. To select the pipeline at compile-time the new env variable VTR_PIPELINE has been added. Use the options LIDAR, RADAR, VISION, RADAR-LIDAR to select your preferred pipeline.
  2. The default dockerfile uses the GPU with a full open-CV build. The OpenCV build is slow, but the hassle of removing the dependency is too high.
  3. Config files exist for the common configurations of the robots. Testing was performed on the Warthog robot. Other robots will require new config files.

@lisusdaniil lisusdaniil merged commit 8fd5f12 into main Feb 27, 2024
1 check passed
@lisusdaniil lisusdaniil deleted the vision_migration branch February 27, 2024 15:29
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.

Upgraded CUDA Support in CMake
5 participants