Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Melodic devel staging #617

Merged
merged 161 commits into from
Nov 10, 2022
Merged

Melodic devel staging #617

merged 161 commits into from
Nov 10, 2022

Conversation

fmauch
Copy link
Contributor

@fmauch fmauch commented Nov 8, 2022

I believe that melodic-devel-staging is elaborate enough to be merged into melodic-devel.

Ludovic Delval and others added 30 commits September 16, 2019 17:36
* moved ur_e_description meshes files to ur_description
* one for each models
* removed each model specific xacro
* use of yaml files
* pass yaml files as parameters
* common ur_robo macro
* remved ur_gazebo specific parts
* use of yaml files parameters
* added e_series
* create a common launch file to avoir duplicated
* default in melodic
Urdf with args and yaml configuration
the kinematics parameters can be retrieved from a calibration mechanism
to precisely represent the robot's kinematics.
This model is only correct in the kinematics structure. Meshes and dynamics
parameters have to be corrected.
Align them with other ROS-I support packages.
avoid deprecation warnings.
Align with other models. For #265.
Align them with other ROS-Industrial support packages.
ipa-nhg and others added 13 commits September 29, 2021 15:13
readme: there are no Kinetic dev jobs any more
…588)

With the common xacro file the robot name will always be "ur".
However, the moveit config packages expect the robot name to be e.g. "ur10e".
This commit will use the macros containing the model name already.
Update MoveIt! configurations to new description structure.

Co-authored-by: Luke Dennis <luke.j.dennis@gmail.com>
Co-authored-by: gavanderhoorn <g.a.vanderhoorn@tudelft.nl>
Co-authored-by: RobertWilbrandt <wilbrandt@fzi.de>
The visual and collision mesh of the base of the ur5 has to be moved 3 mm upwards in order to close the gap between the shoulder and align its bottom in the xy-plane. Otherwise the ur5 collides with the surface it is directly placed on.
In 9d02cca robot launchfiles were forwarded
to their specialized launchfiles instead of setting the robot model in the
load_ur.launch file.
* Update README according to current package state

* Added a note about ur_modern_driver for CB1 and CB2
In order to make the simulation more consistent to the ur_robot_driver
(and ur_modern_driver) this adds a tool0_controller frame.
For the driver this corresponds to the tool transformation published by
the robot controller directly. As this doesn't exist for the simulated
robot, this adds the frame using an identity transform from tool0.
* Add ur_kinematics parameter sets for all e-Series robots

* Add a README with a notice to ur_kinematics

* Set correct IKFast plugin name in kinematics.yaml

* Add ignore files for releasing ur_kinematics

Since the state of this package questionable at the current point, let's
disable releasing it for now.
@fmauch fmauch added this to the melodic-devel-staging milestone Nov 8, 2022
melodic.ignored Outdated
@@ -0,0 +1 @@
ur_kinematics
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn't this have to be added to the release repository?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Those should go to the release repository instead
Copy link
Collaborator

@RobertWilbrandt RobertWilbrandt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With #619 merged all breaking changes should now be done and there is no more reason to delay a release.

@fmauch fmauch merged commit 9238b2c into melodic-devel Nov 10, 2022
@fmauch
Copy link
Contributor Author

fmauch commented Nov 10, 2022

Not deleting the source branch, since there are many references on that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants