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

Add leg skin spatial calibration #897

Merged
merged 10 commits into from
Aug 4, 2023
Merged

Add leg skin spatial calibration #897

merged 10 commits into from
Aug 4, 2023

Conversation

BoPot
Copy link
Contributor

@BoPot BoPot commented Jul 31, 2023

The calibrations are created from provided CAD models of upper and lower leg covers. Source code and files can be found here: https://gitlab.fel.cvut.cz/body-schema/icub/icub-skin-calibration/-/tree/main/skin_models

CStransformations and CSlabels .npy files contain transformations to triangle centers extracted from CAD files and corresponding triangle IDs, respectively. skin_from_CS.py uses these files and .ini files from https://github.com/robotology/icub-main/tree/master/app/skinGui/conf/skinGui to generate taxel position files (visualization included).
calibrations folder contains the position files and read_calib.py script for visualization.

@matejhof @rustlluk @traversaro @maggia80 @fiorisi

@CLAassistant
Copy link

CLAassistant commented Jul 31, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@pattacini pattacini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot @BoPot for your contribution 🚀

I would require you to apply a few changes before merging.

Also, a couple of questions:

app/skinGui/conf/positions/left_leg_lower.txt Outdated Show resolved Hide resolved
@pattacini pattacini linked an issue Jul 31, 2023 that may be closed by this pull request
@pattacini
Copy link
Member

I have edited the names in the headers according to the convention.
Sorry about the link issues in the comment, I have fixed that, too, they should work now.

Thanks a lot 👍🏻

I was not aware of the references in GUI conf files, I will try to add them where needed.

I meant those files in https://github.com/robotology/icub-main/tree/master/app/skinGui/conf that serve to configure the GUI. Some of them do refer to files in the positions subdir.

However, address my request only if you have somehow tested your positions within the GUI as well.

Added position files for legs.
Added position files for legs.
Added the rest of position files.
Added torso position file.
Add leg position file references to conf files
@matejhof
Copy link
Contributor

matejhof commented Aug 3, 2023

The file https://github.com/robotology/icub-main/blob/master/app/skinGui/conf/skinManAll.ini does not include the legs which, given the name of the file and the fact that iCub now comes with skin on the legs, is strange. Shall we consider adding the leg guis and position files to this? Or rename the file?

@pattacini
Copy link
Member

Hi @matejhof

Shall we consider adding the leg guis and position files to this? Or rename the file?

Yep, we can extend the file to include the new positions.

Also, on second thought, I think it'd be valuable to add a README.md file within the positions directory with just a brief section telling how the new .txt files have been generated (i.e., with a pointer to your Gitlab project).

Better off making this information explicit rather than leaving it only in this PR.

@matejhof
Copy link
Contributor

matejhof commented Aug 4, 2023

Ok, let's do that. Maybe we can do it with @rustlluk - test the new skinManAll.ini on the robot first, add the README, and then extend the PR with these changes or make a new PR. It will not happen in the next 2 or 3 weeks though. In the meantime, shall we proceed with the current PR?

@pattacini
Copy link
Member

Agree @matejhof 👍🏻

I can add the README myself ex-post: it's very easy.
I'm going thus to merge this PR.

Thanks for the nice addition!

@pattacini pattacini merged commit 5957d5e into robotology:master Aug 4, 2023
4 checks passed
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.

skinGui positions files for iCub legs
5 participants