Skip to content
This repository was archived by the owner on Jan 14, 2023. It is now read-only.

Action generation #70

Merged
merged 2 commits into from
Aug 31, 2018
Merged

Action generation #70

merged 2 commits into from
Aug 31, 2018

Conversation

exo-core
Copy link
Contributor

@exo-core exo-core commented Aug 30, 2018

I had to make some more adjustments to make the generated messages compatible with actionlib.

There are two messages generated per action part (goal, result, feedback):

  1. A message based on the according section of the .action file (e.g. <ACTION_NAME>Goal)
  2. A wrapper that contains the previous message as well as some meta-information (Header and GoalId/GoalStatus). This on is then named <ACTION_NAME>ActionGoal and used for the communication.

I was now able to run a successful test with an action server based on ernestmc's rosjava_actionlib on our KUKA Sunrise cabinet (robot controller) together with a actionlib axclient.py on my PC. 😄

Copy link

@jubeira jubeira left a comment

Choose a reason for hiding this comment

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

Thanks for the submission @exo-core!
I only have one comment regarding the package version; otherwise LGTM. I'm glad you could make it work with this patch!

package.xml Outdated
@@ -1,7 +1,7 @@
<?xml version="1.0"?>
<package>
<name>rosjava_bootstrap</name>
<version>0.3.2</version>
<version>0.3.3</version>
Copy link

Choose a reason for hiding this comment

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

Would you please remove this change? We usually bump the version in a separate commit with a short description in the changelog.

If this code is already working for you, I can do that right after this PR.

@exo-core
Copy link
Contributor Author

exo-core commented Aug 31, 2018

Reverted that commit.

Thought that new feature was worth increasing the number a little 😜

@jubeira jubeira merged commit fe41a5d into rosjava:kinetic Aug 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants