Weather vane orientation detection

cvra / robot-software

CVRA monorepo - All software running on our bots lives here

MIT License

43 stars 21 forks source link

Weather vane orientation detection #261

Closed antoinealb closed 2 years ago

antoinealb commented 4 years ago

Tested for real.
Sends result over UDP/Protobuf directly into a messagebus topic
Needs some more flexible config, but already sweet sweet proof of concept.

Collaboration brought to you by @nuft and I.

SyrianSpock commented 4 years ago

I don't get the error on CI, operator* should perform matrix multiplication 🤔

msplr commented 4 years ago

I don't get the error on CI, operator* should perform matrix multiplication

I had the same compilation error on the RasPi. Need to investigate...

antoinealb commented 4 years ago

The versions of OpenCV are different between macOS (4.2.0) and Ubuntu Bionic (3.2.0), that could be a source of headaches.

Now I am wondering if we should perhaps build OpenCV as part of the build ?

msplr commented 4 years ago

We should build against the same version. I'm for the more recent one. Also I would specify the version requirement in the CMakeLists.txt so that it fails if not satisfied. Though, I'm not sure if automatically pulling the source and building it here is the best way to do it.