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

Updates state_publisher to robot_state_publisher in the franka_visualization package #137

Conversation

rickstaa
Copy link
Contributor

This commit removes the deprecated 'state_publisher' alias and replaces it with the new 'robot_state_publisher'. See
ros/robot_state_publisher#87.

@domire8
Copy link

domire8 commented Jun 22, 2021

Would be great if this fix could go in! Thanks a lot

@rickstaa rickstaa force-pushed the fix_franka_visualization_robot_state_publisher_error branch from baa0328 to 44a0cdd Compare August 6, 2021 08:37
@rickstaa
Copy link
Contributor Author

rickstaa commented Aug 6, 2021

@domire8 I just rebased it onto the noetic-devel so that the @frankaemika team can easily merge it.

Maverobot pushed a commit that referenced this pull request Sep 8, 2021
…very_behavior to develop

* commit '6172dcfde9dc8ff9cc9a947d2064d947a35cb047':
  fix typo in doc
  cmake version 3.1.3 in meta package franka_ros for melodic and noetic support
  make recovery print debug
  reset command buffer on recover, keep updating while in reflex
  remove redundant . in print
  revert cmake version update in meta package to fix build issue with catkin_make
  only execute controller update when not in error state
@gollth
Copy link
Contributor

gollth commented Sep 14, 2021

Hi @rickstaa,

don't worry, we didn't forget you ;) Thanks in advance for all your effort which you put into making franka_ros a better product. We will have a look at all of your PRs in the near future.

Before we merge anything, though, please be a little bit more patient until we have a proper CI set up with Github actions.

@rickstaa
Copy link
Contributor Author

@gollth No problem, thanks for having such an active development and allowing pull requests. Take your time for my research, I'm currently working from my own branches.

@rickstaa
Copy link
Contributor Author

@gollth Also feel free to tag me if you need information or changes on any of my pull request.

@gollth gollth changed the base branch from noetic-devel to develop September 15, 2021 12:35
@gollth
Copy link
Contributor

gollth commented Sep 15, 2021

Could you rebase this branch to latest develop please?

@rickstaa rickstaa force-pushed the fix_franka_visualization_robot_state_publisher_error branch from 44a0cdd to 860ae7b Compare September 15, 2021 13:48
@rickstaa
Copy link
Contributor Author

@gollth done.

@rickstaa
Copy link
Contributor Author

O develop branch. I now rebased to the noetic-devel branch.

This commit removes the depricated 'state_publisher' alias and replaces it
with the new 'robot_state_publisher'. See
ros/robot_state_publisher#87.
@rickstaa rickstaa force-pushed the fix_franka_visualization_robot_state_publisher_error branch from 860ae7b to e5405c9 Compare September 15, 2021 13:58
@rickstaa
Copy link
Contributor Author

@gollth Now it is rebased onto the develop branch.

@gollth gollth merged commit b9b60e8 into frankaemika:develop Sep 15, 2021
@gollth
Copy link
Contributor

gollth commented Sep 15, 2021

Our CI seems to work now =) I tested on melodic and looks good. Thanks for fixing!

@rickstaa rickstaa deleted the fix_franka_visualization_robot_state_publisher_error branch September 15, 2021 15:12
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.

3 participants