Deprecate accidental leaking of private API

ros / urdf_parser_py

Standalone URDF parser for Python.

BSD 3-Clause "New" or "Revised" License

86 stars 48 forks source link

Open EricCousineau-TRI opened 6 years ago

EricCousineau-TRI commented 6 years ago

Relates #30, #14

It's hard to know what you can really change in the API since some of the internal implementation bits leaked out into the public namespace.

Ideally, it'd be nice to deprecate those private bits, and simplify the public interface, and do a bit more testing on it.

\cc @clalancette

eacousineau commented 6 years ago

Proposed steps:

[ ] Ensure that urdf.py (and sdf.py) do not transitively import symbols from xml_reflection, and try to handle class variabes.
[ ] Deprecate xml_reflection as a whole.
[ ] Once deprecation period has passed, delete deprecated members.