Skip to content

Docker setup#33

Merged
ramjayakumar21 merged 2 commits into
masterfrom
docker-setup
Nov 1, 2025
Merged

Docker setup#33
ramjayakumar21 merged 2 commits into
masterfrom
docker-setup

Conversation

@jasonstaker

@jasonstaker jasonstaker commented Oct 25, 2025

Copy link
Copy Markdown
Member

Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.

  • Resolved dependency linking issues, mainly th integration of ArenaSDK and missing system libraries (pthread, m) needed by MAVLink helper functions to eliminate the linker error w/ MavLink fn's.
  • Ensured all necessary external libraries, including GeographicLib and MetavisionSDK, are located and linked via CMake for a clean/reliable build.
  • Fixed a seg fault by modifying application logic to properly handle the --fake camera mode, guaranteeing the executable runs offline without needing physical hardware.

Resolves #32

Type of change

What types of changes does your code introduce to this project?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

docker build . -t camerafeed:local
docker run --rm camerafeed:local ./camerafeed --seconds 10 --fake
docker run --rm camerafeed:local ./camerafeed --seconds 10 --fake

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. This is simply a reminder of what we are going to look for before merging your code.

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@ramjayakumar21 ramjayakumar21 merged commit 4688fe4 into master Nov 1, 2025
2 checks passed
vayer2005 pushed a commit that referenced this pull request Nov 5, 2025
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.

Dockerize Imaging Pipeline (ArenaSDK only compatible with Linux)

3 participants