robotics-in-concert / rocon_multimaster

Key components for ros multimaster systems
11 stars 19 forks source link

Implement Actions #44

Closed stonier closed 11 years ago

stonier commented 11 years ago

They're just a bundle of pubsubs, shouldn't be too hard (famous last words).

stonier commented 11 years ago
piyushk commented 11 years ago

registering pubs/subs may not be enough (I've done this already). I think the callbacks in actionlib don't get fired correctly on occasion. Trying to figure out when this happens.

stonier commented 11 years ago

I started looking at this last night as well - I don't think I've gotten as far as you did and have not committed anything yet though. Do you have attempts as yet uncommitted?

I could have a look at this if you're stalled, or I could switch to work in parallel getting more remote gateway info on the redis server.

piyushk commented 11 years ago

Mostly what I've done is removed the empty stubs with the corresponding pubs/subs. Advertise and Pull are behaving as expected (tested manually through advertise_all and pull_all, automatically through all the calls). All of this has gotten merged, and I have nothing uncomitted.

Unfortunately, something is still amiss. Let's say you start a client and then pull in the server. The client's callbacks don't seem to fire. On the other hand, if you pull in the server and then start the client -- it seems to work. I haven't worked out what cases work or not, but actionlib depends on message callbacks, some of which may not be firing appropriately through the gateway paradigm.

I won't be able to take a look at this for the next day or two. If you want to take a look at it in this time, then go ahead. I might get back to this later when I try to do some multi-master control of the robots.

stonier commented 11 years ago

Ok, I'll take a look tonight.

stonier commented 11 years ago

I managed to get flips working, will look at pulls tomorrow. Didn't seem to run in to the same problems you had though. Some observations:

stonier commented 11 years ago

Is the actionlib_tutorials averaging server/client actually working? It's bombastic for me. Testing only with the fibonacci now.

piyushk commented 11 years ago

I only tested through fibonacci.

On Wed, Oct 31, 2012 at 9:56 AM, Daniel Stonier notifications@github.comwrote:

Is the actionlib_tutorials averaging server/client actually working? It's bombastic for me. Testing only with the fibonacci now.

— Reply to this email directly or view it on GitHubhttps://github.com/robotics-in-concert/rocon_multimaster/issues/44#issuecomment-9947716.