Support #1248

Add new type 'PointCloudSet3DFloat' to the sandbox

Added by C. Emmerich over 11 years ago. Updated over 11 years ago.

Status:ResolvedStart date:11/20/2012
Priority:NormalDue date:
Assignee:J. Moringen% Done:

100%

Category:Type Proposal
Target version:-

Description

Please add to rst0.7

0001-added-new-proto-type-sets-of-point-clouds.patch Magnifier (977 Bytes) C. Emmerich, 11/20/2012 12:32 PM

0002-changed-documentation-of-PointCloudSets3DFloat-for-c.patch Magnifier (1.17 KB) C. Emmerich, 11/20/2012 03:51 PM

Associated revisions

Revision dd64ae46
Added by C. Emmerich over 11 years ago

Added geometry.PointCloudSets3DFloat: sets of point clouds

refs #1248

Signed-off-by: Jan Moringen <>

Revision 1637b109
Added by C. Emmerich over 11 years ago

Added geometry.PointCloudSets3DFloat: sets of point clouds

fixes #1248

Signed-off-by: Jan Moringen <>

History

#1 Updated by J. Moringen over 11 years ago

Should the new type be added to the master branch?

Also, please modify the documentation string along the lines of

/**
 * A set of 3D Point clouds.
 *
 * @author NAME <EMAIL>
 */

Please comment the clouds field like this:

/**
 * Empty collection of clouds is (allowed|not allowed).
 * The order of cloud objects (is|is not) significant.
 */
repeated geometry.PointCloud3DFloat clouds = 1;

Please submit a new patch with these modifications. Sorry for the inconvenience.

#2 Updated by C. Emmerich over 11 years ago

yes, for backward compatibility it should also be included into master.

New patch is attached.

#3 Updated by J. Moringen over 11 years ago

  • Category set to Type Proposal
  • Status changed from New to In Progress
  • Assignee set to J. Moringen

#4 Updated by C. Emmerich over 11 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

Applied in changeset rst-proto|commit:1637b109f5d1ca71dd3f8a1306a08b5cc7738cf8.

Also available in: Atom PDF