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

Stream and visualize human effort data in HumanStateVisualizer #385

Merged
merged 16 commits into from
Mar 27, 2024

Conversation

dariosortino
Copy link
Collaborator

@dariosortino dariosortino commented Mar 25, 2024

As per title, this PR:

  • Allows the stream and the visualization of human effort data by the color of spheres object
  • Allows to set the maxEffort in the .ini file for each joint to which the associated sphere saturates in red
  • Allows to set a background color from the .ini file

This PR depends by robotology/idyntree#1163

@S-Dafarra

@dariosortino dariosortino changed the title Stream and visualize human effort data to HumanStateVisualizer Stream and visualize human effort data in HumanStateVisualizer Mar 25, 2024
@S-Dafarra
Copy link
Contributor

I think it is worth specifying that you need version 11.1.0 of iDynTree in

otherwise it will fail to compile

@S-Dafarra
Copy link
Contributor

I think it is worth specifying that you need version 11.1.0 of iDynTree in

otherwise it will fail to compile

Actually, 12.0.0 after robotology/idyntree@f5c4bd6 😅

@dariosortino
Copy link
Collaborator Author

@traversaro the checks failed because there were no iDynTree 12.0.0 version upstream, but now that you merged this it should work. How can we trigger the checks again?

@S-Dafarra
Copy link
Contributor

@traversaro the checks failed because there were no iDynTree 12.0.0 version upstream, but now that you merged this it should work. How can we trigger the checks again?

You can rerun failed jobs here:
image

@S-Dafarra
Copy link
Contributor

Actually, this line needs to be edited

@S-Dafarra
Copy link
Contributor

In Windows and MacOS it will fail until idyntree gets updated in conda. See conda-forge/idyntree-feedstock#81

@dariosortino
Copy link
Collaborator Author

I was suspecting something related to conda packages not being updated yet.

You can rerun failed jobs here: image

By the way, I think I don't have the permissions to re-run jobs because I can't see that button

@traversaro
Copy link
Member

By the way, I think I don't have the permissions to re-run jobs because I can't see that button

We can look into fixing this, but just FYI a common workaround is just to close and reopen the PR.

@traversaro
Copy link
Member

You should have permissions now, and iDynTree 12.0.0 should have been released in conda-forge.

@traversaro
Copy link
Member

traversaro commented Mar 26, 2024

Unfortunately here we also install wearables, that still required iDynTree 11. We can switch to build wearables from source, etherwise I guess it could take some days to get wearables compiled with idyntree 12 . To be pragmatic we can also just disable the Windows/macOS CI until iDynTree 12 has been propagated.

@dariosortino
Copy link
Collaborator Author

@traversaro it's up to you, I don't know the details about the different options

@traversaro
Copy link
Member

@traversaro it's up to you, I don't know the details about the different options

Ok, let's just disable macos and windows for now, we can re-enable them in few days.

@dariosortino dariosortino merged commit c618c30 into robotology:master Mar 27, 2024
1 check passed
@dariosortino dariosortino deleted the update-humanstateviz branch March 27, 2024 13:14
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

Successfully merging this pull request may close these issues.

3 participants