uwrobotics / uwrt_mars_rover

Ros Metapackage for UWRT Mars Rover
MIT License
18 stars 7 forks source link

Reorganize Repo into smaller metapackages #102

Closed wmmc88 closed 3 years ago

wmmc88 commented 4 years ago

We want to clean up the organization of our ROS repository:

bold is metapkg. This will be tackled in smaller prs.


wmmc88 commented 4 years ago

@azumnanji @hamzaMahdi @akeaveny @younesr1 any thoughts or additions?

akeaveny commented 4 years ago

Where will the msgs sit? Also, I have uwrt_arm_rover_simulation pkg in uwrt_arm rn.

wmmc88 commented 4 years ago

@akeaveny what does the sim package contain? could it just be merged into a top level sim @azumnanji ?

i think for msgs, if we're doing this smaller metapkg thing, we have msgs split into their respective subsystem metapkg

akeaveny commented 4 years ago

The sim pkg contains some basic launch files for gazebo config (e.g. spawn for x, y, yaw, etc..).

azumnanji commented 4 years ago

sim pkg could be at top level - should have no effect on aidan's current package

wmmc88 commented 3 years ago

@azumnanji is it ok to have a standalone arm urdf in uwrt_mars_rover/uwrt_mars_rover_arm/uwrt_mars_rover_arm_description and then pull in that urdf in the top level description pkg (uwrt_mars_rover/uwrt_mars_rover_description)?

wmmc88 commented 3 years ago

layout for this issue has been merged in #108. leaving this issue pinned for now as a reference for developing the packages (some are just empty placeholder folders rn.)