ros-industrial / universal_robot

ROS-Industrial Universal Robots support (https://wiki.ros.org/universal_robot)
1.12k stars 1.04k forks source link

Documentation: ur_description package structure #539

Open Victorlouisdg opened 4 years ago

Victorlouisdg commented 4 years ago

Hi, I was trying to understand the structure of the ur_description package (of the melodic-devel-staging branch) so I made this diagram to help. I thought it might be useful for other people so maybe we could add this to a readme file that's placed inside the ur_description folder?

ur_description_package_structure

gavanderhoorn commented 4 years ago

Thanks. This would indeed make a nice addition.

Note though that the structure may change, so we might have to update the diagram a few times.

hariharan382 commented 4 years ago

one doubt: in universal_robot folder there are urX_moveit_config(X be some numer) is it moveit config folder or we should generate the moveit configuration file through moveit "setup_assistant" please answer this asap

gavanderhoorn commented 4 years ago

one doubt: in universal_robot folder there are urX_moveit_config(X be some numer) is it moveit config folder or we should generate the moveit configuration file through moveit "setup_assistant"

The packages you refer to are indeed MoveIt configuration packages. I would have expected the moveit_config part in the name to be clear indication of that (it's customary to name MoveIt configuration pkgs for specific robots like thi).

please answer this asap

Please don't do this. We're all volunteers here, helping a community in our free time.

I'm going to hide your (and my) comment, as they are off-topic for this issue. That won't remove them, only not show them by default.

hariharan382 commented 4 years ago

then i generated a package in moveit but it is completely different from already moveit config in your universal_robot package

On Wed, Nov 4, 2020 at 10:39 PM G.A. vd. Hoorn notifications@github.com wrote:

one doubt: in universal_robot folder there are urX_moveit_config(X be some numer) is it moveit config folder or we should generate the moveit configuration file through moveit "setup_assistant"

The packages you refer to are indeed MoveIt configuration packages. I would have expected the moveit_config part in the name to be clear indication of that (it's customary to name MoveIt configuration pkgs for specific robots like thi).

please answer this asap

Please don't do this. We're all volunteers here, helping a community in our free time.

I'm going to hide your (and my) comment, as they are off-topic for this issue. That won't remove them, only not show them by default.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ros-industrial/universal_robot/issues/539#issuecomment-721858545, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALJKNFWDBZJUSSKORZH5RHDSOGDEBANCNFSM4RYBCOFA .

gavanderhoorn commented 4 years ago

Yes, that's why I wrote you shouldn't do that, unless you have experience and would be able to edit the new configuration to work with the files we have here in this repository.

In any case: this is still off-topic for this issue. Please keep your discussion confined to #549.

hariharan382 commented 4 years ago

mam please i just want to move the bot by slightly by python node in rviz,just help me for that

On Thu, Nov 5, 2020 at 5:11 PM G.A. vd. Hoorn notifications@github.com wrote:

Yes, that's why I wrote you shouldn't do that, unless you have experience and would be able to edit the new configuration to work with the files we have here in this repository.

In any case: this is still off-topic for this issue. Please keep your discussion confined to #549 https://github.com/ros-industrial/universal_robot/issues/549.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ros-industrial/universal_robot/issues/539#issuecomment-722325342, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALJKNFSSKLNQNDPE5LD3TGLSOKFNRANCNFSM4RYBCOFA .

Simon248 commented 3 years ago

You can assign it to me.