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

Supported pinocchio with cppadcg installation #180

Merged

Conversation

cmastalli
Copy link
Collaborator

This enables us to install example-robot-data when Pinocchio has been installed with CppADCogen

@hrp2-14
Copy link

hrp2-14 commented Jul 30, 2023

Hi ! This project doesn't usually accept pull requests on the main branch.
If this wasn't intentionnal, you can change the base branch of this PR to devel
(No need to close it for that). Best, a bot.

@cmastalli cmastalli changed the base branch from master to devel July 30, 2023 10:13
Copy link
Collaborator Author

@cmastalli cmastalli left a comment

Choose a reason for hiding this comment

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

Fixed cmake format

@cmastalli cmastalli merged commit 2a878dc into Gepetto:devel Jul 30, 2023
2 checks passed
@cmastalli cmastalli deleted the topic/pinocchio-with-cppad-installation branch July 30, 2023 10:24
@nim65s
Copy link
Member

nim65s commented Aug 1, 2023

This should be fixed in pinocchio packaging, not here, and in each package dependent on pinocchio

@cmastalli
Copy link
Collaborator Author

This should be fixed in pinocchio packaging, not here, and in each package dependent on pinocchio

Agreed! I created an issue and compromise to handle it when is possible. Is this enough for now? Or would you prefer to revert this PR?

nim65s added a commit to nim65s/robotpkg that referenced this pull request Dec 1, 2023
    ## [4.0.9] - 2023-11-29

    ### What's Changed
    * Supported pinocchio with cppadcg installation by cmastalli in Gepetto/example-robot-data#180
    * [pre-commit.ci] pre-commit autoupdate by pre-commit-ci in Gepetto/example-robot-data#181
    * Verbose by nim65s in Gepetto/example-robot-data#183
    * [pre-commit.ci] pre-commit autoupdate by pre-commit-ci in Gepetto/example-robot-data#184
    * [pre-commit.ci] pre-commit autoupdate by pre-commit-ci in Gepetto/example-robot-data#187
    * [pre-commit.ci] pre-commit autoupdate by pre-commit-ci in Gepetto/example-robot-data#188
    * CMake: require >= 3.10 by nim65s in Gepetto/example-robot-data#185
    * Update jrlcmakemodules to support Python 3.12 by jorisv in Gepetto/example-robot-data#190
    * [pre-commit.ci] pre-commit autoupdate by pre-commit-ci in Gepetto/example-robot-data#189
    * [pre-commit.ci] pre-commit autoupdate by pre-commit-ci in Gepetto/example-robot-data#191
    * Sync submodule cmake by jcarpent in Gepetto/example-robot-data#192

    ### New Contributors
    * jorisv made their first contribution in Gepetto/example-robot-data#190
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.

None yet

3 participants