Add CoM info as the output of HumanStateProvider

[kouroshD]

The changes have been tested using offline dumped data.

[yeshasvitirupachuri]

LGTM 👍

@lrapetti if you agree, we can merge it upstream. Let me know.

[yeshasvitirupachuri]

@diegoferigo I think I do not have access to Codacy settings on the branches.
Please double check that and enable it for the branch this PR is open to.
Also, I think we need to update the CI steps to consider the latest changes on wearables too.
Thanks.

[traversaro]

Also, I think we need to update the CI steps to consider the latest changes on wearables too.

The error is:

Cloning into 'wearables'...
error: pathspec 'feature/cleanup' did not match any file(s) known to git.
The command "docker exec testContainer ./.ci/build_deps.sh" failed and exited with 1 during . 

I guess it should be sufficient to remove/change the line in https://github.com/robotology/human-dynamics-estimation/blob/integration_based_teleoperation/.ci/build_deps.sh#L71 .

[lrapetti]

Also, I think we need to update the CI steps to consider the latest changes on wearables too.

The error is:

Cloning into 'wearables'...
error: pathspec 'feature/cleanup' did not match any file(s) known to git.
The command "docker exec testContainer ./.ci/build_deps.sh" failed and exited with 1 during . 

I guess it should be sufficient to remove/change the line in https://github.com/robotology/human-dynamics-estimation/blob/integration_based_teleoperation/.ci/build_deps.sh#L71 .

It has to be changed with feature/ICub-device-impl. @kouroshD can you please add this change to the PR?

[traversaro]

It has to be changed with feature/ICub-device-impl. @kouroshD can you please add this change to the PR?

Alternatively, you can just fix it with a PR on the target branch integration_based_teleoperation (that looks a lot like a devel branch :D ), and then rebase all the PRs.

[kouroshD]

Also, I think we need to update the CI steps to consider the latest changes on wearables too.

The error is:

Cloning into 'wearables'...
error: pathspec 'feature/cleanup' did not match any file(s) known to git.
The command "docker exec testContainer ./.ci/build_deps.sh" failed and exited with 1 during . 

I guess it should be sufficient to remove/change the line in https://github.com/robotology/human-dynamics-estimation/blob/integration_based_teleoperation/.ci/build_deps.sh#L71 .

It has to be changed with feature/ICub-device-impl. @kouroshD can you please add this change to the PR?

done!

[yeshasvitirupachuri]

Also, I think we need to update the CI steps to consider the latest changes on wearables too.

The error is:

Cloning into 'wearables'...
error: pathspec 'feature/cleanup' did not match any file(s) known to git.
The command "docker exec testContainer ./.ci/build_deps.sh" failed and exited with 1 during . 

I guess it should be sufficient to remove/change the line in https://github.com/robotology/human-dynamics-estimation/blob/integration_based_teleoperation/.ci/build_deps.sh#L71 .

It has to be changed with feature/ICub-device-impl. @kouroshD can you please add this change to the PR?

@lrapetti why feature/ICub-device-impl branch? master branch of wearables is stable to use right ?

[kouroshD]

Also, I think we need to update the CI steps to consider the latest changes on wearables too.

The error is:

Cloning into 'wearables'...
error: pathspec 'feature/cleanup' did not match any file(s) known to git.
The command "docker exec testContainer ./.ci/build_deps.sh" failed and exited with 1 during . 

I guess it should be sufficient to remove/change the line in https://github.com/robotology/human-dynamics-estimation/blob/integration_based_teleoperation/.ci/build_deps.sh#L71 .

It has to be changed with feature/ICub-device-impl. @kouroshD can you please add this change to the PR?

@lrapetti why feature/ICub-device-impl branch? master branch of wearables is stable to use right ?

Cuurently, the integration_based_teleoperation branch of HDE is compatible with feature/ICub-device-impl of wearables. If I am not wrog the same applies for the master branch of HDe as well.

[kouroshD]

@lrapetti If everything is fine, please follow with merging.

[lrapetti]

@lrapetti If everything is fine, please follow with merging.

Thanks @kouroshD
Let's wait for the checks and then I will merge it

[lrapetti]

The check for ubuntu 18.04 passed, while Travis failed for Ubuntu 16.04 that is not meant to be supported for HDE (I will provide removing the tests).
Apveyor instead has been broken for long time, and it can be removed for the moment.
I will procede with merging then.

[lrapetti]

integration_based_teleoperation has been moved to a new devel branch