added native sensor_msgs::PointCloud2 support without conversion
Since the conversion from PointCloud2 to PointCloud and back for assembling the clouds is bad, I added a native sensor_msgs::PointCloud2 support without conversion. I also added an organized mode, to assemble PointClouds2 to an organized PointCloud2.
overall very useful, thank you. It will probably take use a few days to get through the code. I believe some code should be part of the base class (as private lightweight functions). E.g., all that code is duplicated for PointCloud and PointCloud2:
if (field->offset != this->scan_hist_[start_index].fields[chan_ind].offset){
Please have only one function for those (like a function that return analyses two different PointFields)
@spuetz , any comments on that ?
@vrabaud, I added the method hasEqualScanHeader to check the headers. as recommended by you. Additionally, I fixed the down-sampling for the organized mode.
@vrabaud, would be nice, if we can complete this pull request and merge it. thx