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

Pinocchio requires lower numpy version #349

Merged
merged 1 commit into from
Sep 3, 2024

Conversation

hello-binit
Copy link
Contributor

@hello-binit hello-binit commented Aug 25, 2024

Using the latest 1.x version of numpy (1.26.4) is not possible with Pinoochio, a dependency of Web Teleop, because it depends on cmeel-boost, which depends on numpy 1.24.x. In a previous PR, we increased the numpy version from 1.23.x to make trimesh happy. To keep both happy, this commit pins on >=1.24

Follow up to PR #348

Using the latest 1.x version of numpy (1.26.4) is not possible
with Pinoochio, a dependency of Web Teleop, because it depends
on cmeel-boost, which depends on numpy 1.24.x. In a previous
PR, we increased the numpy version from 1.23.x to make trimesh
happy. To keep both happy, this commit pins on >=1.24
hello-binit added a commit to hello-robot/stretch_web_teleop that referenced this pull request Aug 25, 2024
This commits removes the pin on an older version of numpy. Stretch Body now pins the right version of numpy. Details in these two PRs:
 - hello-robot/stretch_body#349
 - hello-robot/stretch_body#348
@hello-binit hello-binit merged commit 2b340af into master Sep 3, 2024
@hello-binit hello-binit deleted the bugfix/pinocchio_numpy_pin branch September 3, 2024 21:48
hello-binit added a commit to hello-robot/stretch_web_teleop that referenced this pull request Sep 4, 2024
* Remove numpy pin in Web Teleop

This commits removes the pin on an older version of numpy. Stretch Body now pins the right version of numpy. Details in these two PRs:
 - hello-robot/stretch_body#349
 - hello-robot/stretch_body#348

* Numpy primitives deprecated

With numpy 1.24 and onwards, using np.bool and other np.<primitive>
types is deprecated, and raises exceptions. We are supposed to use
the built-in primitives (i.e. bool, float, etc.).
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.

1 participant