RobotLocomotion / drake

Model-based design and verification for robotics.
https://drake.mit.edu
Other
3.37k stars 1.27k forks source link

Consolidated home for visualization components #19153

Open jwnimmer-tri opened 1 year ago

jwnimmer-tri commented 1 year ago

It's not very easy to locate our visualization component for any given entity. I think we should move all of the components into one package so they are easy to browse and understand:

These C++ components should move into drake/visualization:

Likewise their python bindings should move into pydrake.visualization.

These are widely used, so it will be three steps:

(1) Move the modules and add forwarding shims to the old names continue to work. Ensure the old (shimmed) names do not appear in any API reference docs, examples, tutorials, etc.

(2) Introduce deprecation warnings. Maybe at T+6 months.

(3) Remove the forwarding shims. Maybe at T+12 months.

DamrongGuoy commented 1 year ago

It sounds really good to me. I didn't even know some of those files before. Thanks!

jwnimmer-tri commented 1 year ago

In the interim, we should probably ensure that all of those classes are tagged into Doxygen: https://drake.mit.edu/doxygen_cxx/group__visualization.html

jwnimmer-tri commented 1 month ago

Duplicate of #19786.