Fix python syntax in matlab-whole-body-simulator multisheller scripts and switch ergocub-software to use conda package from conda-forge #1639
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
traversaro
changed the title
There was a problem, new run: https://github.com/robotology/robotology-superbuild/actions/runs/8817941386 . |
xela-95
approved these changes
Apr 24, 2024
|
Now the build failed with: This is not easy to interpreter, but I guess the problem is that we are trying to pin to an old version of ipopt and a new version of idyntree, and there is no new version of idyntree compatible with such an old version of ipopt. Let's see if there is something to clean up. |
Indeed, the problem was an old workaround: 6b666fc . New job: https://github.com/robotology/robotology-superbuild/actions/runs/8819849615 . |
New failure: The problem is probably related to conda-forge/vtk-feedstock#321 . To be honest, I think this is just a false problem, as we are inserting expat as a dependency in a lot of packages even if it is not actually necessary. Accordingly to this to permit to blocktest to compile fine, but I am not sure why that should be the case. I think we can try to remove it and test again. |
While this could be a nice improvement, note that it can be still problematic due to #1607 (comment) . |
|
Cool, removing the However, we can track this in a separate issue, as this does not block uploading new version of the packages. Actually, probably we can wait a day or two to check if conda-forge/staged-recipes#26175 is merged, so that we can avoid building ergocub-software in robotology channel. |
traversaro
changed the title
|
ergocub-software was moved to conda-forge, let's remove the corresponding multisheller scripts and add the appropriate parameter to signal to the conda package generation machinery that the ergocub-software from conda-forge should be used. |
traversaro
changed the title
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 #1634 .
Test run of the conda package generation CI scripts: https://github.com/robotology/robotology-superbuild/actions/runs/8817191771 .