Skip to content

Commit

Permalink
fixed a bug in BRISK keypoint estimation where if the keypoint type
Browse files Browse the repository at this point in the history
would be different than PointWithScale a compiler error would occur
  • Loading branch information
rbrusu committed Mar 29, 2013
1 parent 7531527 commit a421a88
Showing 1 changed file with 3 additions and 10 deletions.
13 changes: 3 additions & 10 deletions keypoints/include/pcl/keypoints/impl/brisk_2d.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -77,16 +77,9 @@ pcl::BriskKeypoint2D<PointInT, PointOutT, IntensityT>::detectKeypoints (PointClo

pcl::keypoints::brisk::ScaleSpace brisk_scale_space (octaves_);
brisk_scale_space.constructPyramid (image_data, width, height);
// Check if the template types are the same. If true, avoid a copy.
// The PointOutT MUST be registered using the POINT_CLOUD_REGISTER_POINT_STRUCT macro!
if (isSamePointType<PointOutT, pcl::PointWithScale> ())
brisk_scale_space.getKeypoints (threshold_, output.points);
else
{
pcl::PointCloud<pcl::PointWithScale> output_temp;
brisk_scale_space.getKeypoints (threshold_, output_temp.points);
pcl::copyPointCloud<pcl::PointWithScale, PointOutT> (output_temp, output);
}
pcl::PointCloud<pcl::PointWithScale> output_temp;
brisk_scale_space.getKeypoints (threshold_, output_temp.points);
pcl::copyPointCloud<pcl::PointWithScale, PointOutT> (output_temp, output);

// we do not change the denseness
output.width = int (output.points.size ());
Expand Down

0 comments on commit a421a88

Please sign in to comment.