Feature #1294
Add BoundingBox3D and BoundingBox3DSet to the sandbox
Status: | Resolved | Start date: | 12/11/2012 | |
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | - | % Done: | 100% | |
Category: | Type Proposal | |||
Target version: | Robotics Service Bus - rsb-0.9 |
Description
please add to both trunk and 0.7
Related issues
Associated revisions
Added types BoundingBox3DFloat[Set] in proto/sandbox/rst/geometry/
fixes #1294
- proto/sandbox/rst/geometry/BoundingBox3DFloat.proto: new file; 3D
bounding box in general position and orientation - proto/sandbox/rst/geometry/BoundingBox3DFloatSet.proto: new file;
set of BoundingBox3DFloat
Signed-off-by: Jan Moringen <jmoringe@techfak.uni-bielefeld.de>
Backport: Added types BoundingBox3DFloat[Set] in proto/sandbox/rst/geometry/
refs #1294
- proto/sandbox/rst/geometry/BoundingBox3DFloat.proto: new file; 3D
bounding box in general position and orientation - proto/sandbox/rst/geometry/BoundingBox3DFloatSet.proto: new file;
set of BoundingBox3DFloat
Signed-off-by: Jan Moringen <jmoringe@techfak.uni-bielefeld.de>
History
#1 Updated by J. Moringen over 11 years ago
- Status changed from New to Feedback
I propose the modified version in 0001-Added-types-BoundingBox3DFloat-Set-in-proto-sandbox-.patch
.
In addition, I suggest an axis-aligned bounding box (#1295).
Please comment.
#2 Updated by J. Moringen over 11 years ago
#3 Updated by J. Moringen over 11 years ago
- Category set to Type Proposal
- Target version set to rsb-0.9
#4 Updated by J. Wienke over 11 years ago
What is the motivation to specify the center of mass with the transformation? Wouldn't it be easier to understand if one of the corners was used?
#5 Updated by C. Emmerich over 11 years ago
Actually not (in my opinion). Specifying a corner in 3 or even in N-D and in presence of or before a rotation seems not to be that intuitive. In contrast, the center of mass is - at least before the rotation - very easy to understand (no formula needed or soemthing else; we only deal with boxes).
I talked to Johannes about that and he agreed.
#6 Updated by Anonymous over 11 years ago
Looks good to me. I'm in favor of jmoringe's adaption since API should be easier to access (and to understand) if lengths are more explicit (width, height, depth).
#7 Updated by C. Emmerich over 11 years ago
- Status changed from Feedback to Resolved
- % Done changed from 0 to 100
Applied in changeset rst-proto|commit:0fbffc97f24c608483952a332c33bea29a3ff420.