Skip to content

Fix syncer bug causing wrong frame order #8378

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Feb 23, 2021
Merged

Fix syncer bug causing wrong frame order #8378

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
a49d873
1. fixed l500 matcher hierarchy
aangerma Feb 17, 2021
6d05f45
CR fixes
aangerma Feb 18, 2021
c832dcf
CR fixes
aangerma Feb 18, 2021
0a3fe1c
Fix for Linux
aangerma Feb 18, 2021
2eba69e
CR fixes
aangerma Feb 21, 2021
bde0e82
Fixes
aangerma Feb 21, 2021
85eaf25
Fix
aangerma Feb 21, 2021
043ffc9
More CR fixes
aangerma Feb 21, 2021
0cd6dd3
Fixed small bug
aangerma Feb 22, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 19 additions & 19 deletions src/sync.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -533,28 +533,28 @@ namespace librealsense
LOG_DEBUG("by_pass_composite_matcher: " << _name << " " << frame_to_string(f));
_callback(std::move(f), env);
}
std::string frame_to_string(const frame_holder & f)

std::string frame_to_string( const frame_holder & f )
{
std::ostringstream s;
auto composite = dynamic_cast< composite_frame * >( f.frame );
if( composite )
{
std::ostringstream s;
auto composite = dynamic_cast<composite_frame*>(f.frame);
if (composite)
{
for (int i = 0; i < composite->get_embedded_frames_count(); i++)
{
auto frame = composite->get_frame(i);
s << frame->get_stream()->get_stream_type() << " " << frame->get_frame_number() << " " << std::fixed << frame->get_frame_timestamp() << " ";
}
}
else
for( int i = 0; i < composite->get_embedded_frames_count(); i++ )
{
s << f->get_stream()->get_stream_type();
s << " " << f->get_stream()->get_unique_id();
s << " " << f->get_frame_number();
s << " " << std::fixed << (double)f->get_frame_timestamp();
s << " ";
auto frame = composite->get_frame( i );
s << frame->get_stream()->get_stream_type() << " " << frame->get_frame_number()
<< " " << std::fixed << frame->get_frame_timestamp() << " ";
}
return s.str();
}
else
{
s << f->get_stream()->get_stream_type();
s << " " << f->get_stream()->get_unique_id();
s << " " << f->get_frame_number();
s << " " << std::fixed << (double)f->get_frame_timestamp();
s << " ";
Copy link
Contributor

@maloel maloel Feb 21, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's this for?

}
return s.str();
}
}
} // namespace librealsense