search

CentroEPiaggio / kuka-lwr

Software related to the KUKA LWR 4+: for real and for simulation.
The Unlicense
101 stars 81 forks source link

Arm state controller merge #52

Closed marcoesposito1988 closed 7 years ago

marcoesposito1988 commented 8 years ago

Basically a rebase of #41 on top of the Merge

carlosjoserg commented 8 years ago

Oh, I hadn't see the last message in #41, the vote for merging.

But the building of this one is failing too, it does not find the kdl_conversions package

Once this is merge is merge I'll close #41

marcoesposito1988 commented 8 years ago

Yes, probably we have to add it to the packages installed through apt in the Travis script.

I'll do it ASAP

carlosjoserg commented 8 years ago

Ok, please, do the travis script update in a separate PR

carlosjoserg commented 7 years ago

I also found some discrepancies in the URDF related to the dynamic properties of links... I will PR the changes so you all can see what I'm talking about and why I believe the estimated values were not that accurate

I already added the kdl conversions to travis btw

carlosjoserg commented 7 years ago

Closing in favor of #74

marcoesposito1988 commented 7 years ago

Cool! Nice to see you're back :) I will try the whole thing out as soon as possible.. probably in around 10 days. After that I may be more active myself again.

carlosjoserg commented 7 years ago

Great!