From b8209bca754eca9fc961e985dba6ec6d51de52ad Mon Sep 17 00:00:00 2001 From: Eran Date: Thu, 26 Sep 2024 12:46:34 +0300 Subject: [PATCH] remove spurious debug log messages no longer needed --- src/context.cpp | 8 ++++---- src/dds/rs-dds-device-proxy.cpp | 10 +++++----- src/dds/rs-dds-sensor-proxy.cpp | 2 +- src/environment.cpp | 2 +- src/proc/formats-converter.cpp | 4 ++-- third-party/realdds/src/dds-device-impl.cpp | 4 ++-- 6 files changed, 15 insertions(+), 15 deletions(-) diff --git a/src/context.cpp b/src/context.cpp index bcd47a95a1d..b02ed8ea618 100644 --- a/src/context.cpp +++ b/src/context.cpp @@ -116,7 +116,7 @@ namespace librealsense { { for( auto & dev_info : factory->query_devices( requested_mask ) ) { - LOG_INFO( "... " << dev_info->get_address() ); + LOG_DEBUG( "... " << dev_info->get_address() ); list.push_back( dev_info ); } } @@ -124,12 +124,12 @@ namespace librealsense { { if( auto dev_info = item.second.lock() ) { - LOG_INFO( "... " << dev_info->get_address() ); + LOG_DEBUG( "... " << dev_info->get_address() ); list.push_back( dev_info ); } } - LOG_INFO( "Found " << list.size() << " RealSense devices (0x" << std::hex << requested_mask << " requested & 0x" - << get_device_mask() << " from device-mask in settings)" << std::dec ); + LOG_DEBUG( "Found " << list.size() << " RealSense devices (0x" << std::hex << requested_mask + << " requested & 0x" << get_device_mask() << " from device-mask in settings)" << std::dec ); return list; } diff --git a/src/dds/rs-dds-device-proxy.cpp b/src/dds/rs-dds-device-proxy.cpp index 0f540a66502..6628d1c8843 100644 --- a/src/dds/rs-dds-device-proxy.cpp +++ b/src/dds/rs-dds-device-proxy.cpp @@ -149,7 +149,7 @@ dds_device_proxy::dds_device_proxy( std::shared_ptr< const device_info > const & , auto_calibrated_proxy() , _dds_dev( dev ) { - LOG_DEBUG( "=====> dds-device-proxy " << this << " created on top of dds-device " << _dds_dev.get() ); + //LOG_DEBUG( "=====> dds-device-proxy " << this << " created on top of dds-device " << _dds_dev.get() ); register_info( RS2_CAMERA_INFO_NAME, dev->device_info().name() ); register_info( RS2_CAMERA_INFO_PHYSICAL_PORT, dev->device_info().topic_root() ); register_info( RS2_CAMERA_INFO_PRODUCT_ID, "DDS" ); @@ -236,8 +236,8 @@ dds_device_proxy::dds_device_proxy( std::shared_ptr< const device_info > const & sensor_info.proxy->add_dds_stream( sidx, stream ); _stream_name_to_owning_sensor[stream->name()] = sensor_info.proxy; type_and_index_to_dds_stream_sidx.insert( { type_and_index, sidx } ); - LOG_DEBUG( sidx.to_string() << " " << get_string( sensor_info.type ) << " '" << stream->sensor_name() - << "' : '" << stream->name() << "' " << get_string( stream_type ) ); + //LOG_DEBUG( sidx.to_string() << " " << get_string( sensor_info.type ) << " '" << stream->sensor_name() + // << "' : '" << stream->name() << "' " << get_string( stream_type ) ); software_sensor & sensor = get_software_sensor( sensor_info.sensor_index ); auto video_stream = std::dynamic_pointer_cast< realdds::dds_video_stream >( stream ); @@ -281,7 +281,7 @@ dds_device_proxy::dds_device_proxy( std::shared_ptr< const device_info > const & for( auto & sensor_info : sensor_name_to_info ) { - LOG_DEBUG( sensor_info.first ); + //LOG_DEBUG( sensor_info.first ); // Set profile's ID based on the dds_stream's ID (index already set). Connect the profile to the extrinsics graph. for( auto & profile : sensor_info.second.proxy->get_stream_profiles() ) @@ -348,7 +348,7 @@ dds_device_proxy::dds_device_proxy( std::shared_ptr< const device_info > const & auto const dds_extr = _dds_dev->get_extrinsics( from_stream.first, to_stream.first ); if( ! dds_extr ) { - LOG_DEBUG( "missing extrinsics from " << from_stream.first << " to " << to_stream.first ); + //LOG_DEBUG( "missing extrinsics from " << from_stream.first << " to " << to_stream.first ); continue; } rs2_extrinsics extr = to_rs2_extrinsics( dds_extr ); diff --git a/src/dds/rs-dds-sensor-proxy.cpp b/src/dds/rs-dds-sensor-proxy.cpp index 4a0d7f0c9a0..f4cfb7caa63 100644 --- a/src/dds/rs-dds-sensor-proxy.cpp +++ b/src/dds/rs-dds-sensor-proxy.cpp @@ -609,7 +609,7 @@ void dds_sensor_proxy::add_option( std::shared_ptr< realdds::dds_option > option if( get_option_handler( option_id ) ) throw std::runtime_error( "option '" + option->get_name() + "' already exists in sensor" ); - LOG_DEBUG( "... option -> " << option->get_name() ); + //LOG_DEBUG( "... option -> " << option->get_name() ); auto opt = std::make_shared< rs_dds_option >( option, [=]( json value ) diff --git a/src/environment.cpp b/src/environment.cpp index a01eec497a3..d597c21a362 100644 --- a/src/environment.cpp +++ b/src/environment.cpp @@ -133,7 +133,7 @@ namespace librealsense } if (!invalid_ids.empty()) - LOG_INFO("Found " << invalid_ids.size() << " unreachable streams, " << std::dec << counter << " extrinsics deleted"); + LOG_DEBUG("Found " << invalid_ids.size() << " unreachable streams, " << std::dec << counter << " extrinsics deleted"); } int extrinsics_graph::find_stream_profile(const stream_interface& p, bool add_if_not_there) diff --git a/src/proc/formats-converter.cpp b/src/proc/formats-converter.cpp index 1e7ee3c2f4b..baf15d7f02e 100644 --- a/src/proc/formats-converter.cpp +++ b/src/proc/formats-converter.cpp @@ -97,7 +97,7 @@ stream_profiles formats_converter::get_all_possible_profiles( const stream_profi for( auto & raw_profile : raw_profiles ) { - LOG_DEBUG( "Raw profile: " << raw_profile ); + //LOG_DEBUG( "Raw profile: " << raw_profile ); for( auto & pbf : _pb_factories ) { const auto & sources = pbf->get_source_info(); @@ -129,7 +129,7 @@ stream_profiles formats_converter::get_all_possible_profiles( const stream_profi target.resolution_transform( width, height ); cloned_vsp->set_dims( width, height ); } - LOG_DEBUG( " -> " << cloned_profile ); + //LOG_DEBUG( " -> " << cloned_profile ); // Cache pbf supported profiles for efficiency in find_pbf_matching_most_profiles _pbf_supported_profiles[pbf.get()].push_back( cloned_profile ); diff --git a/third-party/realdds/src/dds-device-impl.cpp b/third-party/realdds/src/dds-device-impl.cpp index 9fc6aefe516..6dd10f1511f 100644 --- a/third-party/realdds/src/dds-device-impl.cpp +++ b/third-party/realdds/src/dds-device-impl.cpp @@ -590,7 +590,7 @@ void dds_device::impl::on_device_header( json const & j, dds_sample const & samp { std::string const & from_name = ex[0].string_ref(); std::string const & to_name = ex[1].string_ref(); - LOG_DEBUG( "[" << debug_name() << "] ... got extrinsics from " << from_name << " to " << to_name ); + //LOG_DEBUG( "[" << debug_name() << "] ... got extrinsics from " << from_name << " to " << to_name ); extrinsics extr = extrinsics::from_json( ex[2] ); _extrinsics_map[std::make_pair( from_name, to_name )] = std::make_shared< extrinsics >( extr ); } @@ -700,7 +700,7 @@ void dds_device::impl::on_stream_options( json const & j, dds_sample const & sam dds_options options; for( auto & option_j : options_j ) { - LOG_DEBUG( "[" << debug_name() << "] ... " << option_j ); + //LOG_DEBUG( "[" << debug_name() << "] ... " << option_j ); auto option = dds_option::from_json( option_j ); options.push_back( option ); }