Code review comment for lp:~acerisara/autopilot/move-to-object
Revision history for this message
| Thomi Richards (thomir-deactivatedaccount) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
35a32c5
(Get the code!)
just to give a bit more context:
I don't see any problem with making the 'get_center_point' function part of the public autopilot API, and documenting it as such. Once that's done, you can then change the docstring as I suggested on IRC.
To make get_center_point public:
1) Import it in the autopilot.input package. Make sure it appears in __all__. This should cause the docstring to appear in the auto-generated docs.
2) Change any code that uses that function to import it from autopilot.input, NOT from the private module.
3) profit!