actionlib icon indicating copy to clipboard operation
actionlib copied to clipboard

Published 20 hours ago • verified publisher icon ros

Improve axclient.py

Open Torchtopher opened this issue 2 years ago • 5 comments

As someone who uses axclient fairly regularly, there were quite a few limitations that made it somewhat annoying to use. This pull request improves the functionality of axclient adding:

  • Support for multiple actions within the same window
  • Loads all actionlib servers without having to type the topic name
  • Has message file for viewing when sending actions, useful for enums/comments in the raw message file

Here is what the GUI looks like now. image Now run with just rosrun actionlib_tools axclient.py and the rest is taken care of. This is my first contribution but I wanted to share the changes I had made. Would be happy to make any changes/improvements needed to merge!

Torchtopher avatar Feb 13 '23 20:02 Torchtopher

Not a maintainer, just reviewing voluntarily.

  • Support for multiple actions within the same window

What this means is unclear to me. Does the gui with your change 1) take in multiple different goals, 2) show feedback and result per each, on a single window? That sounds great (while I personally can't think of a usecase), but also makes me worried if the gui could get confusing. Would you mind clarifying?

130s avatar Jun 14 '23 11:06 130s

@130s Yes, it does allow you to send multiple goals and see the feedback for them. The gui is still relatively simple, as you only see the information for a single action at a time and use the Action Selector menu at the top to switch between them. The usecase, at least for us, is that often you need to call multiple actionlib servers, or don't quite know which one you want, and can then look at an overview of all of them in that menu. Let me know if you have any other questions!

Torchtopher avatar Jun 15 '23 21:06 Torchtopher

@mjcarroll @jacobperron You're listed as maintainers IINM. Is this something that can be merged into ros1 branch? I'm inclined to say this includes new features.

Separately from this PR, I'm interested in ros2-fying axclient.py (discussed in https://github.com/ros2/rcl/issues/1079). If there's a chance of this PR getting merged into ros1 then I'll wait for the code to be finalized (to avoid having to repeatedly cherry-pick).

130s avatar Jun 21 '23 10:06 130s

@mjcarroll @jacobperron Any update on this? I would be happy to make any changes needed to get this merged.

Torchtopher avatar Sep 10 '23 23:09 Torchtopher

@Torchtopher great seeing you at FIRST championship. I will add this to my list of things to review this week.

mjcarroll avatar Apr 22 '24 12:04 mjcarroll