LUCID Camera Client

[atar13]

Closes #29

[atar13]

This PR is ready for review. It ended up touching a lot of files across the project so those reviewing can focus on files in include/camera and src/camera.

I can't figure out why the unit tests are failing. It seems to not be able to find the LucidCamera declaration. It's defined in include/camera/lucid.hpp but is behind a #ifdef that is set in the deps/arena.cmake file if Arena SDK is installed. I'll do some more digging but you can run ninja test and ninja obcpp to verify that everything is compiling correctly.

[Tyler-Lentz]

Style wise everything looks good, thanks for putting this together.

I think the tests are failing however? Looks like a compile error with creating a lucid camera so maybe a test is out of date. So once we fix that and verify still works on the jetson I think we should be good

[atar13]

I think the tests are failing however? Looks like a compile error with creating a lucid camera so maybe a test is out of date. So once we fix that and verify still works on the jetson I think we should be good

Yeah that's what this comment was about. I'll try to figure it out:

#159 (comment)

[Tyler-Lentz]

I think the tests are failing however? Looks like a compile error with creating a lucid camera so maybe a test is out of date. So once we fix that and verify still works on the jetson I think we should be good

Yeah that's what this comment was about. I'll try to figure it out:

#159 (comment)

oh sorry i didn't see that

[Samir-Rashid]

The code works. I'm a bit hesitant to merge because there are a lot of must fix problems left. To make sure this doesn't get even more out of hand, we should merge now. I do not want to block work on main that builds on this.

[atar13]

Tests are passing now after some shenanigans with GitHub actions not detecting the environment variable for the ArenaSDK location set in the Docker image.

[Tyler-Lentz]

Thank you for your hard work soldier o7