<!> 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:

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)


Wiki: laser_assembler/Reviews/2009-01-06_API_Review (last edited 2009-09-02 13:21:02 by JeremyLeibs)