Added invalid point checking to RGBD and Depth Camera plugins to set is_dense flag accordingly

[Dyst-0]

🦟 Bug fix

Fixes #574

Summary

Fix similar to #180. Added gz::math::isnan and std::isinf checks to depth/coordinates in all FillMsg methods in PointUtil.cc
, which RGBD Camera and Depth Camera depend on.

Reproducibility steps described in #574.
New output should look the same, except for the is_dense flag, which should be set to false.

(maybe another test/world should be done where the point cloud only contains valid points, to check if is_dense is correctly set to true)

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• Consider updating Python bindings (if the library has them)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers
• Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

[iche033]

Thanks for the fix!

[Dyst-0]

@iche033 should pass the checks now👍

[iche033]

@Mergifyio backport main gz-sensors9 gz-sensors8

[mergify]

backport main gz-sensors9 gz-sensors8

✅ Backports have been created

Details

• #578 Added invalid point checking to RGBD and Depth Camera plugins to set is_dense flag accordingly (backport #575) has been created for branch main
• #579 Added invalid point checking to RGBD and Depth Camera plugins to set is_dense flag accordingly (backport #575) has been created for branch gz-sensors9
• #580 Added invalid point checking to RGBD and Depth Camera plugins to set is_dense flag accordingly (backport #575) has been created for branch gz-sensors8