At the time of this review the package was named point_cloud_assembler
API review
Proposer: Vijay Pradeep
Present at review:
- List reviewers
Question / concerns / comments
Enter your thoughts on the API and any questions / concerns you have here. Please sign your name. Anything you want to address in the API review should be marked down here before the start of the meeting.
Brian:
Homepage link in manifest should point to http://pr.willowgarage.com/wiki/point_cloud_assembler
- Is downsampling uniform or random?
ConvertToCloud and GetPointsInScan should be convertToCloud and getPointsInScan
Tully
- What is the most common case. Should this be a library for latency and bandwidth usage?
- The documentation doesn't explain what it does under the hood. (Message Notifier, how it queues things does it blow things away after it sends them, etc.)
Meeting agenda
To be filled out by proposer based on comments gathered during API review period
Conclusion
Package status change (also mark change on PackageStatusDict page)
Change link
document downsampling look at maybe making it more uniform
Make it into a node which uses a library which does the heavy lifting
Document how it works to allow debugging. (When will it do things, how might things fail)