i/rtabmap_ros
1
0
Fork 0
mirror of https://github.com/introlab/rtabmap_ros.git synced 2026-10-03 16:27:46 +08:00
Code Issues Packages Projects Releases Wiki Activity

deskewing: added console logging, handling case where timestamps are not ordered in lidar rings

Browse Source
This commit is contained in:
matlabbe
2023-10-01 08:52:14 -07:00
parent f38bf6c267
commit 223ba62b1c
1 changed files with 101 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
rtabmap_conversions/src/MsgConversion.cpp
+101 -10
View File
@@ -2673,17 +2673,103 @@ bool deskew_impl(
ros::Time lastStamp; ros::Time lastStamp;
if(timeDatatype == 6) // UINT32 if(timeDatatype == 6) // UINT32
{ {
unsigned int nsec = *((const unsigned int*)(&input.data[0]+offsetTime)); unsigned int nsecFirst = *((const unsigned int*)(&input.data[0]+offsetTime));
firstStamp = input.header.stamp+ros::Duration(0, nsec); unsigned int nsecLast = *((const unsigned int*)(&input.data[(input.width-1)*input.point_step + input.row_step*(input.height-1)]+offsetTime));
nsec = *((const unsigned int*)(&input.data[timeOnColumns?(input.width-1)*input.point_step:(input.height-1)*input.row_step]+offsetTime));
lastStamp = input.header.stamp+ros::Duration(0, nsec); if(nsecFirst > nsecLast)
{
// scans are not ordered, we need to search min/max
static bool warned = false;
if(!warned) {
UWARN("Timestamp channel is not ordered, we will have to parse every scans to "
"determinate first and last time offsets. This will add slightly computation "
"time. This warning is only shown once.");
warned = true;
}
if(timeOnColumns)
{
for(size_t i=0; i<input.width; ++i)
{
unsigned int nsec = *((const unsigned int*)(&input.data[(i)*input.point_step]+offsetTime));
if(nsec < nsecFirst)
{
nsecFirst = nsec;
}
else if(nsec > nsecLast)
{
nsecLast = nsec;
}
}
}
else
{
for(size_t i=0; i<input.height; ++i)
{
unsigned int nsec = *((const unsigned int*)(&input.data[input.row_step*(i)]+offsetTime));
if(nsec < nsecFirst)
{
nsecFirst = nsec;
}
else if(nsec > nsecLast)
{
nsecLast = nsec;
}
}
}
}
firstStamp = input.header.stamp+ros::Duration(0, nsecFirst);
lastStamp = input.header.stamp+ros::Duration(0, nsecLast);
} }
else if(timeDatatype == 7) // FLOAT32 else if(timeDatatype == 7) // FLOAT32
{ {
float sec = *((const float*)(&input.data[0]+offsetTime)); float secFirst = *((const float*)(&input.data[0]+offsetTime));
firstStamp = input.header.stamp+ros::Duration().fromSec(sec); float secLast = *((const float*)(&input.data[(input.width-1)*input.point_step + input.row_step*(input.height-1)]+offsetTime));
sec = *((const float*)(&input.data[timeOnColumns?(input.width-1)*input.point_step:(input.height-1)*input.row_step]+offsetTime));
lastStamp = input.header.stamp+ros::Duration().fromSec(sec); if(secFirst > secLast)
{
// scans are not ordered, we need to search min/max
static bool warned = false;
if(!warned) {
UWARN("Timestamp channel is not ordered, we will have to parse every scans to "
"determinate first and last time offsets. This will add slightly computation "
"time. This warning is only shown once.");
warned = true;
}
if(timeOnColumns)
{
for(size_t i=0; i<input.width; ++i)
{
float sec = *((const float*)(&input.data[(i)*input.point_step]+offsetTime));
if(sec < secFirst)
{
secFirst = sec;
}
else if(sec > secLast)
{
secLast = sec;
}
}
}
else
{
for(size_t i=0; i<input.height; ++i)
{
float sec = *((const float*)(&input.data[input.row_step*(i)]+offsetTime));
if(sec < secFirst)
{
secFirst = sec;
}
else if(sec > secLast)
{
secLast = sec;
}
}
}
}
firstStamp = input.header.stamp+ros::Duration().fromSec(secFirst);
lastStamp = input.header.stamp+ros::Duration().fromSec(secLast);
} }
else else
{ {
@@ -2691,9 +2777,14 @@ bool deskew_impl(
return false; return false;
} }
if(lastStamp <= firstStamp) if(lastStamp < firstStamp)
{ {
ROS_ERROR("First and last stamps in the scan are the same!"); ROS_ERROR("Last stamp (%f) is smaller than first stamp (%f) (header=%f)!", lastStamp.toSec(), firstStamp.toSec(), input.header.stamp.toSec());
return false;
}
else if(lastStamp == firstStamp)
{
ROS_ERROR("First and last stamps in the scan are the same (%f) (header=%f)!", lastStamp.toSec(), input.header.stamp.toSec());
return false; return false;
} }