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

Error importing pyluos.modules (pyluos version 1.4.1) #66

Closed
agillies8 opened this issue Dec 12, 2020 · 1 comment
Closed

Error importing pyluos.modules (pyluos version 1.4.1) #66

agillies8 opened this issue Dec 12, 2020 · 1 comment
Labels
bug Something isn't working

Comments

@agillies8
Copy link

Reachy package cannot import pylous.modules. Likely because most recent version of pylous (1.4.1) renamed modules to containers.

To reproduce:

from reachy import parts, Reachy

r = Reachy(
    right_arm=parts.RightArm(io='ws', hand='force_gripper'),
    left_arm=parts.LeftArm(io='ws', hand='force_gripper'),
)

To fix:

Change line 8 in luos.py to:

from pyluos.containers import DynamixelMotor

@agillies8 agillies8 added the bug Something isn't working label Dec 12, 2020
@pierre-rouanet
Copy link
Member

Hi @agillies8,

Thanks for reporting this issue!
It's indeed due to recent API changes in pyluos.
The newest version is actually incompatible with the module version we used. I thus recommend to keep pyluos 1.2.* for the moment.
I've updated the setup.py file accordingly.

We'll update the pyluos version in our next release.

@pierre-rouanet pierre-rouanet changed the title Error importing pyluos.modules Error importing pyluos.modules (pyluos version 1.4.1) Dec 14, 2020
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