Gold856

Results 165 comments of Gold856

Would it be possible to add support for struct/protobuf decoding? It would be much easier to use the API if decoding was built-in instead of something the user has to...

The ArmSimulation tests should be added back.

After checking frc-docs, the only command examples that are RLIed and won't be replaced by wpilibsuite/frc-docs#2684 or wpilibsuite/frc-docs#2698 are DriveDistanceOffboard, both of the Hatchbot examples, and SelectCommand. DriveDistanceOffboard has also...

USBCamera doesn't give a Mat by itself, it's instead added as a source to a CvSink, and the Mat is pulled from that. We could instead use a RawSink, which...

Looks like you're going to have to patch the user config header to not have `#cmakedefine`. How you handle that is up to you (use an ifndef and a define...

Probably want to update this to use the latest commit of Catch2 as well.

I think the solution to the config header is to just clone the repo, configure the build with CMake, and copy the generated header into our source tree. Bonus points...

We frequently patch libraries under `thirdparty` directories to make them fit our needs. Typically, we manage patches with [upstream_utils](https://github.com/wpilibsuite/allwpilib/blob/main/upstream_utils/README.md), but it looks like MemoryBuffer, the only consumer of MapMode, [is...

#1396 removed CalibDB and there seems to have been a docs change made separately so that CalibDB is just not even mentioned. Closing as fixed.

I like the single file per project version. I think that keeping everything together is better than having several short build files scattered throughout the repo, since it makes it...