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

ur_rtde's prebuilt wheels are not compatible with Python 3.11 #121

Closed
m-decoster opened this issue Feb 1, 2024 · 2 comments
Closed

ur_rtde's prebuilt wheels are not compatible with Python 3.11 #121

m-decoster opened this issue Feb 1, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@m-decoster
Copy link
Contributor

m-decoster commented Feb 1, 2024

Describe the bug

When trying to use airo-robots from a conda environment with Python 3.11, the import of rtde_control fails:

File "~/.conda/envs/airo-mono/lib/python3.11/site-packages/airo_robots/manipulators/hardware/ur_rtde.py", line 10, in <module>
    from rtde_control import RTDEControlInterface
ModuleNotFoundError: No module named 'rtde_control'

This is a known issue: https://gitlab.com/sdurobotics/ur_rtde/-/issues/294

Apparently, building the wheel from source fixes this, but this requires a CMake installation.

I'm raising this issue because airo-mono's environment.yaml file specifies Python 3.11 (though it works fine on 3.10). I discovered it because I was trying to use some functionality that only became available in Python 3.11.

To reproduce

  1. Create and activate a new conda environment with Python 3.11 and pip as dependencies (see below)
  2. Run pip install ur_rtde
  3. Within the python REPL, run import rtde_control
name: temp-environment
dependencies:
  - python=3.11
  - pip

Possible solutions

Use Python 3.10, build ur_rtde from source, or wait for this issue to be fixed to use Python 3.11.

@m-decoster m-decoster added the bug Something isn't working label Feb 1, 2024
@m-decoster m-decoster self-assigned this Feb 1, 2024
@tlpss
Copy link
Contributor

tlpss commented Feb 2, 2024

@m-decoster nice catch, maybe best if we revert the python version to 3.10 for now and wait for the binaries to be pushed to PyPI? Feel free to make that change if you agree :)

m-decoster added a commit that referenced this issue Feb 5, 2024
m-decoster added a commit that referenced this issue Feb 5, 2024
* Revert Python to version 3.10 #121

* Keep 3.11 in the testing matrix
@m-decoster
Copy link
Contributor Author

I can confirm that the underlying issue has been fixed. Nothing needs to be done on our end, but it's now possible again to use Python 3.11 with ur_rtde.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants