DDS Y16 streaming is mono16 for ROS2 compatibility

OhadMeir · Jan 13, 2025 · bb1b76d · bb1b76d
1 parent d40c2e2
commit bb1b76d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/third-party/realdds/py/pyrealdds.cpp b/third-party/realdds/py/pyrealdds.cpp
@@ -829,7 +829,7 @@ PYBIND11_MODULE(NAME, m) {
         .def( "__repr__", []( dds_video_encoding const & self ) { return self.to_string(); } );
     video_encoding.attr( "z16" ) = dds_video_encoding( "16UC1" );
     video_encoding.attr( "y8" ) = dds_video_encoding( "mono8" );
-    video_encoding.attr( "y16" ) = dds_video_encoding( "Y16" );  // todo should be mono16
+    video_encoding.attr( "y16" ) = dds_video_encoding( "mono16" );
     video_encoding.attr( "byr2" ) = dds_video_encoding( "BYR2" );
     video_encoding.attr( "yuyv" ) = dds_video_encoding( "yuv422_yuy2" );
     video_encoding.attr( "uyvy" ) = dds_video_encoding( "uyvy" );

diff --git a/third-party/realdds/src/dds-stream-profile.cpp b/third-party/realdds/src/dds-stream-profile.cpp
@@ -80,7 +80,7 @@ int dds_video_encoding::to_rs2() const
         { "mono8", RS2_FORMAT_Y8 },  // Used by IR streams; ROS2-compatible
         { "Y8I",  RS2_FORMAT_Y8I },
         { "W10",  RS2_FORMAT_W10 },
-        { "Y16",  RS2_FORMAT_Y16 },
+        { "mono16",  RS2_FORMAT_Y16 }, // Used by IR streams; ROS2-compatible
         { "Y12I", RS2_FORMAT_Y12I },
 //      { "Y16I", RS2_FORMAT_Y16I },
         { "16UC1", RS2_FORMAT_Z16 },  // Used by depth streams; ROS2-compatible
@@ -113,7 +113,7 @@ dds_video_encoding dds_video_encoding::from_rs2( int rs2_format )
     case RS2_FORMAT_Y8: encoding = "mono8"; break;
     case RS2_FORMAT_Y8I: encoding = "Y8I"; break;
     case RS2_FORMAT_W10: encoding = "W10"; break;
-    case RS2_FORMAT_Y16: encoding = "Y16"; break;
+    case RS2_FORMAT_Y16: encoding = "mono16"; break;
     case RS2_FORMAT_Y12I: encoding = "Y12I"; break;
 //  case RS2_FORMAT_Y16I: encoding = "Y16I"; break;
     case RS2_FORMAT_Z16: encoding = "16UC1"; break;