API review

Proposer: Brian Gerkey

Present at review:

  • List reviewers

We're reviewing the map format, and the command-line and ROS APIs of map_server and map_saver:

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.

  • Bhaskara
    • Probably more in keeping with current ROS conventions to have the map be a latched message on a topic?

      (./) Added a latched advertisement on the map topic. The service is left in, because it's also convenient, and there's a lot of existing code that uses it. The documentation is updated, with a note that the service may eventually be phased out.

Meeting agenda

To be filled out by proposer based on comments gathered during API review period

Conclusion

Package status change mark change manifest)

  • /!\ Action items that need to be taken.

  • {X} Major issues that need to be resolved


Wiki: map_server/Reviews/2009-09-20 API Review (last edited 2009-09-26 00:01:16 by BrianGerkey)