human-dynamics-estimation icon indicating copy to clipboard operation
human-dynamics-estimation copied to clipboard

Replace usage of `analogServer`

Open lrapetti opened this issue 2 years ago • 17 comments

Currently we are using the analogServer in order to stream the wrench data from IHumanWrench interface (see https://github.com/search?q=repo%3Arobotology%2Fhuman-dynamics-estimation%20analogServer&type=code). However, the analogServer corresponds to the analogWrapper mentioned in https://github.com/robotology/yarp/pull/2999, which will be soon be deprecated.

Indeed we should understand what are the alternatives. One option could be to implement our own message, and wrapper/remapper devices as we are doing for the IHumanState and IHumanDynamics, and removing the IAnalogSensor interface from the wrench provider (see here). Clearly this will also break the ROS pipeline, which anyway should be soon addressed (see also https://github.com/robotology/human-dynamics-estimation/issues/335).

Another option, would be to understand what is the alternative device.

lrapetti avatar Sep 28 '23 10:09 lrapetti

cc @traversaro

lrapetti avatar Sep 28 '23 10:09 lrapetti

For the ROS part, you can use WrenchStampedRosPublisher as long as you expose the data also with a yarp::dev::ISixAxisForceTorqueSensors. For the YARP part, who is using /HDE/HumanWrenchPublisher/wrench:o ? There is any assumption that the data is in vector form?

traversaro avatar Sep 28 '23 12:09 traversaro

For the YARP part, who is using /HDE/HumanWrenchPublisher/wrench:o ? There is any assumption that the data is in vector form?

I think that specific port is not used, but there is a second analogServer here publishing on /HDE/HumanWrenchWrapper/wrench:o. That port is generally used by the visualizer assuming it's a vector (see here). In case we develop dedicated wrapper/remapper (nws/nwc), we can easily change that part and it would probably also improve the readability. I don't know if you already had in mind another alternative solution.

lrapetti avatar Sep 28 '23 12:09 lrapetti

I have opened https://github.com/robotology/human-dynamics-estimation/issues/361 to address the replacement of analogServer for what concern the YARP messages. It will remain the analogServer to stream data in ROS which should be replaced.

lrapetti avatar Sep 28 '23 15:09 lrapetti

I think we can close this issue, am I right?

claudia-lat avatar May 10 '24 10:05 claudia-lat

Apparently three files are still using that:

  • https://github.com/search?q=repo%3Arobotology%2Fhuman-dynamics-estimation%20analogServer%20&type=code

I would remove the files before closing the issue. If instead the files are still used, then we have a problem. fyi @S-Dafarra @mebbaid

traversaro avatar May 10 '24 11:05 traversaro

Apparently three files are still using that:

  • https://github.com/search?q=repo%3Arobotology%2Fhuman-dynamics-estimation%20analogServer%20&type=code

I would remove the files before closing the issue. If instead the files are still used, then we have a problem. fyi @S-Dafarra @mebbaid

I think this is similar to what @dariosortino did for the iFeel part. Those wrappers were useful to stream over ROS the values of the shoes I suppose. I doubt they work anyhow, since the ROS group has disappeared from recent versions of the device.

S-Dafarra avatar May 10 '24 12:05 S-Dafarra

As far as I know

  • on the human.xml side, the main users are the iFeel human wrenches visualization guys. I think they already moved from streaming the data in that way @dariosortino may confirm.
  • The PHRI XML files were used by @lrapetti for his experiments. Those same experiments will be reproduced by @CarlottaSartore and in this case it maybe a good chance to remove them and adapt to new versions of the device

mebbaid avatar May 10 '24 16:05 mebbaid

  • on the human.xml side, the main users are the iFeel human wrenches visualization guys. I think they already moved from streaming the data in that way @dariosortino may confirm.

Sorry but about this I don't really know much what's going on currently

dariosortino avatar May 13 '24 06:05 dariosortino

  • on the human.xml side, the main users are the iFeel human wrenches visualization guys. I think they already moved from streaming the data in that way @dariosortino may confirm.

Sorry but about this I don't really know much what's going on currently

I guess it all boils down to which files you are using in the iFeel-Human demos. If you are not using https://github.com/robotology/human-dynamics-estimation/blob/48d0147a69572cb2d3c3cf06c421d2703e0bc2ed/conf/xml/Human.xml#L197 anymore, then I guess we can remove them. fyi @claudia-lat @GiulioRomualdi

traversaro avatar May 13 '24 06:05 traversaro

Checking in our configuration files, it seems that we don't use that anymore

dariosortino avatar May 13 '24 07:05 dariosortino

Perfect, I guess we can delete the human.xml in this repo then.

traversaro avatar May 13 '24 07:05 traversaro

  • The PHRI XML files were used by @lrapetti for his experiments. Those same experiments will be reproduced by @CarlottaSartore and in this case it maybe a good chance to remove them and adapt to new versions of the device

Those file are actually older, I think they were used for An.Dy demo before I even joined! Concerning Human.xml that's the file that was used for Xsens, I don't know if it might still be needed.

lrapetti avatar May 13 '24 07:05 lrapetti

It would be nice also to correct this https://github.com/robotology/human-dynamics-estimation/blob/master/misc/hde_scheme.png

dariosortino avatar May 13 '24 09:05 dariosortino

It would be nice also to correct this https://github.com/robotology/human-dynamics-estimation/blob/master/misc/hde_scheme.png

@lrapetti do you happen to know where the source of that drawing is?

traversaro avatar May 13 '24 09:05 traversaro

@traversaro I've got the .svg of the scheme, I will fix it together with @davidegorbani.

claudia-lat avatar May 13 '24 10:05 claudia-lat

you can use the updated figure discussed in the handover in https://github.com/ami-iit/element_ergonomy-control/issues/183#issuecomment-1872496045

mebbaid avatar May 13 '24 10:05 mebbaid