Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge wearables in human-dynamics-estimation? #212

Open
traversaro opened this issue Sep 19, 2024 · 6 comments
Open

Merge wearables in human-dynamics-estimation? #212

traversaro opened this issue Sep 19, 2024 · 6 comments

Comments

@traversaro
Copy link
Member

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 Investigate generation of conda binaries for linux-aarch64 robotology-superbuild#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
Copy link

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
Copy link
Member Author

traversaro commented Sep 24, 2024

The plan is:

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

@traversaro
Copy link
Member Author

* [ ]  Package human-dynamics-estimation 4.0.0 in conda-forge (currently blocked by [Add recipe for icub-firmware-shared packages conda-forge/staged-recipes#27675](https://github.com/conda-forge/staged-recipes/pull/27675) and [Add recipe for icub-main packages conda-forge/staged-recipes#27692](https://github.com/conda-forge/staged-recipes/pull/27692))

In progress in conda-forge/staged-recipes#27723 .

@traversaro
Copy link
Member Author

* [ ]  Package human-dynamics-estimation 4.0.0 in conda-forge (currently blocked by [Add recipe for icub-firmware-shared packages conda-forge/staged-recipes#27675](https://github.com/conda-forge/staged-recipes/pull/27675) and [Add recipe for icub-main packages conda-forge/staged-recipes#27692](https://github.com/conda-forge/staged-recipes/pull/27692))

In progress in conda-forge/staged-recipes#27723 .

PR ready, just waiting for review now.

@traversaro
Copy link
Member Author

PR merged, the last step is to swich the conda generation pipeline for robotology channel to use the new packages, see robotology/robotology-superbuild#1727 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants