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

hector_gazebo: 0.5.2-1 in 'noetic/distribution.yaml' [bloom] #27797

Merged
merged 2 commits into from
Dec 18, 2020

Conversation

StefanFabian
Copy link
Contributor

Increasing version of package(s) in repository hector_gazebo to 0.5.2-1:

hector_gazebo

  • No changes

hector_gazebo_plugins

* Add case for rotating in place in odometry (#75 <https://github.com/tu-darmstadt-ros-pkg/hector_gazebo/issues/75>)
  * Add case for rotating in place in odometry
  Odometry calculations were not taking in account the case of the robot rotating in place. This
  would cause a division by zero and make the plugin return invalid data. This commit handles this
  case by not using the arc method if the distance traveled by the robot is close to zero.
  * Fix negative Y-axis motion bug
  Only the displacement along X and the combined displacement were used to
  calculate the angle between the X-axis and the robot drive direction.
  This resulted in a positive angle whatever if the displacement along Y
  was positive or negative. This commit addresses the issue by using both
  displacements along X and Y to calculate the angle.
* Allow publishing sensor_msgs/MagneticField (#73 <https://github.com/tu-darmstadt-ros-pkg/hector_gazebo/issues/73>)
  Add a new useMagneticFieldMsgs parameter to the magnetometer plugin, allowing the node to publish sensor_msgs/MagneticField messages instead of std_msgs/Vector3Stamped
* Add support of holonomic robot by adding missing Y component in odometry (#71 <https://github.com/tu-darmstadt-ros-pkg/hector_gazebo/issues/71>)
  * Add support of holonomic robot by adding missing Y component in odometry
  The odometry was only computed taking in account linear velocity on the x-axis and angular
  velocity around the z-axis. The code is modified to include linear velocity on the y-axis for
  support of holonomic robots. Odometry arc and radius are calculated the same way as before but
  using the combined linear velocity instead. The end position is then rotated by the angle between
  the combined velocity vector and the x-axis
  * Change variable name
  * Rename combined velocity angle variable
* Merge pull request #64 <https://github.com/tu-darmstadt-ros-pkg/hector_gazebo/issues/64> from ms-iot/upstream_windows_fix
  [Windows][melodic-devel] Follow catkin guide to update the install path
* fix install path. (#1 <https://github.com/tu-darmstadt-ros-pkg/hector_gazebo/issues/1>)
* Merge pull request #53 <https://github.com/tu-darmstadt-ros-pkg/hector_gazebo/issues/53> from enwaytech/ml/cmp0054_fix
  add CMP0045 cmake policy before gazebo include in CMakeLists.txt
* add CMP0045 cmake policy before gazebo include in CMakeLists.txt
* Contributors: Chris I-B, Johannes Meyer, Matthias Loebach, RobinB, Sean Yen, Stefan Fabian

hector_gazebo_thermal_camera

* Merge pull request #64 <https://github.com/tu-darmstadt-ros-pkg/hector_gazebo/issues/64> from ms-iot/upstream_windows_fix
  [Windows][melodic-devel] Follow catkin guide to update the install path
* fix install path. (#1 <https://github.com/tu-darmstadt-ros-pkg/hector_gazebo/issues/1>)
* Merge pull request #53 <https://github.com/tu-darmstadt-ros-pkg/hector_gazebo/issues/53> from enwaytech/ml/cmp0054_fix
  add CMP0045 cmake policy before gazebo include in CMakeLists.txt
* add CMP0045 cmake policy before gazebo include in CMakeLists.txt
* Contributors: Johannes Meyer, Matthias Loebach, Sean Yen

hector_gazebo_worlds

  • No changes

hector_sensors_gazebo

  • No changes

Copy link
Contributor

@mabelzhang mabelzhang left a comment

Choose a reason for hiding this comment

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

Thanks! We were looking for this in Noetic :)

Same comment as in #27794 regarding the LICENSE file. If this PR were for a new package, we would require that. Please see the section about the LICENSE file here https://github.com/ros/rosdistro/blob/master/REVIEW_GUIDELINES.md#types-of-pull-requests

@mabelzhang mabelzhang merged commit 63a1f21 into ros:master Dec 18, 2020
@StefanFabian
Copy link
Contributor Author

No problem :)
This depends on #27798 though via #27796. Just for your information because it won't build otherwise.
That dependency has always been there but it was just recently specified as part of our clean up efforts.

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.

2 participants