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

EoInertials3: refactor in order to make it work also w/ ems and mtb3. #439

Merged
merged 9 commits into from
Nov 23, 2023

Conversation

Nicogene
Copy link
Member

@Nicogene Nicogene commented Nov 13, 2023

This PR integrates in EoInertials3 the functionalities of EoInertials2. In this way EoInertials3 can be runned for retreiving the imu data both from legacy boards such as ems mtb3 AND rfe, mtb4, strain2.

It requires:

@Nicogene Nicogene self-assigned this Nov 13, 2023
@Nicogene Nicogene changed the title WIP refactor eotheinertials3 EoInertials3: refactor in order to make it work also w/ ems and mtb4. Nov 13, 2023
@Nicogene Nicogene changed the title EoInertials3: refactor in order to make it work also w/ ems and mtb4. EoInertials3: refactor in order to make it work also w/ ems and mtb3. Nov 13, 2023
@Nicogene Nicogene force-pushed the refactorEOInertials3 branch from 79295bf to a63cb5e Compare November 15, 2023 15:02
@Nicogene Nicogene force-pushed the refactorEOInertials3 branch from 837928f to 3044c0f Compare November 23, 2023 09:05
Nicogene added a commit to robotology/icub-firmware-build that referenced this pull request Nov 23, 2023
Built w/ icub-firmware-shared@9ba514d46c013ebe5855a80be1321346d814e8b2 and icub-firmware@ffb06fc2d9febb6948f196d0c582d5c9cf071d12

See robotology/icub-firmware#439
@Nicogene Nicogene marked this pull request as ready for review November 23, 2023 09:27
@Nicogene Nicogene requested a review from marcoaccame November 23, 2023 09:29
Copy link
Contributor

@marcoaccame marcoaccame left a comment

Choose a reason for hiding this comment

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

ok, thanks

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.

2 participants