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

control pose/velocity vs measured pose/velocity and tf #220

Open
bmegli opened this issue Mar 17, 2024 · 10 comments
Open

control pose/velocity vs measured pose/velocity and tf #220

bmegli opened this issue Mar 17, 2024 · 10 comments

Comments

@bmegli
Copy link

bmegli commented Mar 17, 2024

From:

it looks like the data available through ROS is control pose/velocity, not the measured pose/velocity.

  1. Is it still the case? (looks it was at end of 2023)
  2. Do I understand correctly that joint_states are filled from control data (vs measured) and in consequence so is ROS tf data?
  3. If above, what difference should be expected between control/measured pose/velocity from user side?
  4. If above, are there any plans to share measured pose/velocity over ROS?

Kind regards!

@penglongxiang
Copy link
Contributor

Hi, to better assist with your issue, may I ask what is the controlling method you plan to use in ROS? Is it through Moveit or our wrapped ROS services?

@bmegli
Copy link
Author

bmegli commented Mar 18, 2024

Hi @penglongxiang,

MoveIt like, sending controller commands to the trajectory controller topic.

But I am specifically interested in

  • what exactly is published over ROS /tf topic
    • which likely has data from /joint_states

So the question is about time/pose synchronization using /tfwith other subsystem rather then control itself.

Will the control vs measured pose/velocity on ROS side affect us in such scenario etc?

@penglongxiang
Copy link
Contributor

Hi @bmegli, so you are using the ros-controller and the hardware interface to our robot. We have just optimized for this scenario and the /joint_states published will be using measured joint angles, so as the joint velocity. It will be available in the next controller firmware version 2.4.0 which is supposed to be released by the end of this month. /tf result is derived from /joint_states and urdf model, so the source of /tf will also be updated.

BTW if you need less delay or higher update rate, please specify report_type:=dev at launching stage, the publish rate will increase from 5Hz to 100Hz. Please also update the 'xarm_ros' repository to the latest since we will keep optimizing and adding new functions.

@bmegli
Copy link
Author

bmegli commented Mar 19, 2024

Thank you @penglongxiang

From XARM side we are using realMove_exec.launch with xarm6_traj_controller

BTW if you need less delay or higher update rate, please specify report_type:=dev

Yes, we are already doing it as well as bumping jnt_stat_pub_rate to get /tf data at higher frequency

We have just optimized for this scenario and the /joint_states published will be using measured joint angles, so as the joint velocity. It will be available in the next controller firmware version 2.4.0 which is supposed to be released by the end of this month

Ok! In that case I will keep the issue open until new firmware is released

@penglongxiang
Copy link
Contributor

Hi @bmegli, I am sorry to say that our official firmware release will be delayed. If you are eager to test the measured joint angles as feedback, I can send you a beta version that already have this feature early next week.

@bmegli
Copy link
Author

bmegli commented Apr 24, 2024

Thank you @penglongxiang

I think we will wait for the stable version

@bmegli
Copy link
Author

bmegli commented Jun 24, 2024

Has the Hi @bmegli, I am sorry to say that our official firmware release will be delayed. If you are eager to test the measured joint angles as feedback, I can send you a beta version that already have this feature early next week.

Has the new firmware (2.4.0?) with measured pose/velocity/tf been eventually released?

@penglongxiang
Copy link
Contributor

Yes, it has been released already, sorry I forgot to notify you in particular. Please follow the update instruction inside the user manual (Appendix 4) and have a test.

@nitish3693
Copy link

@penglongxiang Do use this feature do we need to use the latest ROS driver from the repository, or does it work with old versions (old commits) from the repository.

@penglongxiang
Copy link
Contributor

@nitish3693 It depends on how old your local commit is, it is recommended to use the latest or recent commits for better compatibility with the new firmware.

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

No branches or pull requests

3 participants