There should be no dependency on the sensor library from the rendering library.
yep, that's the reason it was not ported into 1.3, a bunch of the parent sensor variables (hfov, etc.) that describes the sensor need to be migrated into rendering::GpuLaser class directly. See ticket
by the way, it should be fixed in this patch
If this is not urgent, could you write a unit test?
what's an example of the new unit test infrastructure?
I added unit tests in pull request #239 and pull request #245 that you can look at (SDF_Test.cc and BulletTypes_TEST.cc). Write a test file with the same name as the class with suffix _TEST.cc, then use the cmake macro gz_build_tests in the CMakeLists.txt.
I'll tackle the unit test for this pull request.
Hold off on reviewing this until the threaded sensors pull request is put in.
GPU ray sensor is broken. This pull request needs a lot of love.