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

Networkx should not be fixed to 2.2 in the setup.py #27

Open
Daniella1 opened this issue Sep 23, 2022 · 2 comments
Open

Networkx should not be fixed to 2.2 in the setup.py #27

Daniella1 opened this issue Sep 23, 2022 · 2 comments

Comments

@Daniella1
Copy link

When installing urdfpy using pip, (I tried both from pypi and cloning the repo), an error occurs when loading and showing a URDF file.
I ran the code:

import urdfpy

robot = urdfpy.URDf.load("ur5e/ur5e.urdf")
robot.show()

And received the error:

ImportError: cannot import name 'gcd' from 'fractions'

It seems the issue that the setup.py file requires that networkx==2.2, but that version doesn't support the functions used in the library, and therefore needs to be upgraded after installing the library.
I believe the networkx version requirement can be removed from the setup.py file.
(This was tested with Python version 3.9.12 and version 3.10.6 with same results)

@adamconkey
Copy link

I also had problems with this, to the point I couldn't even do import networkx, it would immediately give an error with something like AttributeError: module 'numpy' has no attribute 'int'. I got around it by doing pip install --upgrade networkx after everything else installed and it fixed the problem. I get an ugly pip warning saying it's now incompatible with urdfpy though, so it'd be nice to remove that hard version dependency.

@michael-projectx
Copy link

Similar problems here as well.

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

No branches or pull requests

3 participants