Merge wearables in human-dynamics-estimation?

traversaro commented 2 months ago

Hello everyone, given that:

Both wearables and human-dynamics-estimation do not have a well define maintainer, with people (mainly me, @S-Dafarra and @dariosortino) handling PRs and/or tags and released when needed.
We would like to have human-dynamics-estimation conda packages with python bindings enabled on linux-aarch64, to test the recent work of team X on Nvidia SoC systems like Jetson, and the easiest way to get this done (see https://github.com/robotology/robotology-superbuild/issues/738) is to bring the related packages in conda-forge.

I thought that to simplify mantainance of the repo, we could merge wearables in human-dynamics-estimation so that:

They can be packaged and mantained as a single package on conda-forge
If we need to do a change and released them, this can be done with a single PR that modify both, and if a release/tag is necessary a single one can be done.

Anyone has any specific opinion on this? If people agree, I will be glad to do the change myself. Note that if there is someone that only need wearables, I plan to add specific CMake options so that you could still build only the existing wearables code out of the human-dynamics-estimation repo.

To give you an idea of the current state of dependencies, this is the current tree of dependencies in robotology-superbuild (generated via dot -Tpdf robotology-superbuild.dot -o view.pdf in the build of the superbuild):

robotology-superbuild

dariosortino commented 2 months ago

I never worked directly on wearables but I think that for iFeel applications is not a problem. Let us know if there is something to update in whatever is used by human-dynamics-estimation and wearables

traversaro commented 2 months ago

PR ready for the migration:

traversaro commented 2 months ago

The plan is:

[x] Get the green light of @dariosortino and @S-Dafarra (see https://github.com/robotology/human-dynamics-estimation/pull/395#issuecomment-2370637902)
[x] Merge https://github.com/robotology/human-dynamics-estimation/pull/395 and https://github.com/robotology/robotology-superbuild/pull/1720
[x] Tag human-dynamics-estimation 4.0.0 (major bump to easily detect before and after the merge)
[x] Package human-dynamics-estimation 4.0.0 in conda-forge (currently blocked by https://github.com/conda-forge/staged-recipes/pull/27675 and https://github.com/conda-forge/staged-recipes/pull/27692)

Once this is done, we should have everything to run the online inference/ML part of Team X demos in conda-forge, including linux-aarch64 packages. fyi @GiulioRomualdi @giotherobot @carloscp3009