Generate again conda package for walking-controllers and fix conda metapackages generation #1583
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
This was referenced Jan 29, 2024
Add periodic build of generate-conda-packages with no upload but dummy metapackages generation
#1584
Open
|
robotology-distro-all generation still fails: |
|
Trying to solve the env: locally I have a more clear error: In a nutshell, for sure we need to bump at least cppad |
|
Fixing cppad and removing walking-controllers, we still have errors: the problems goes away removing the proxsuite pin, so probably it is just a missing rebuilt of casadi with the latest proxsuite. |
Actually it is again a problem on our side, proxsuite should be updated to 0.6.1 . |
But the latest is actually proxsuite 0.6.3, so I also asked for a rebuild on casadi side: conda-forge/casadi-feedstock#100) |
|
While indeed the lagging behind of proxsuite and cppad could be a bit a problem in robotology-superbuild and a bit a lack of pinning for these two libraries in conda-forge-pinning, in general we do not care too much about the version of these libraries, even if they are tracked in latest.releases.yaml . Probably we could consider to just avoid adding them to the version listed in robotology-distro and robotology-distro-all . |
new test build: https://github.com/robotology/robotology-superbuild/actions/runs/7695901115 . |
traversaro
changed the title
Build successful, let's merge. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix #1577 .
In a rush to actually generate conda packages and fix #1572, we disabled walking-controllers generation, as it was failing on Linux. However, the generation was failing as the latest bipedal-locomotion-framework was compiled against ipopt 3.14.14, while icub-main was still building with 3.14.13 . This was happening as:
robotology-superbuild/.github/workflows/generate-conda-packages.yaml
Line 179 in d5c49b8
We fixed this in #1578 by manually pinning ipopt to 3.14.14 in the robotology conda packages generation CI. This can be removed once ipopt 3.14.14 abi migration is completed at the conda-forge level.
To understand how to avoid this in the future:
cc @GiulioRomualdi