search

hungpham2511 / toppra

robotic motion planning library
https://hungpham2511.github.io/toppra/index.html
MIT License
625 stars 170 forks source link

python requirements boken in latest updates #53

Closed pbeeson closed 5 years ago

pbeeson commented 5 years ago

As of 5/17/2019, the top version of scipy in pip depends on python-3.5. It's unclear exactly what platform you are targeting as a default working platform, but if you are still shooting for Python 2.7 then you'll want scipy<1.3 in requirements.txt..

hungpham2511 commented 5 years ago

I have tested toppra on Python 3 and things seem yo work. Can I know the steps you have taken to install?

Btw you can try 'pip install toppra' instead.

Bests,

Hung

From: Patrick Beeson notifications@github.com Sent: Saturday, May 18, 2019 11:25:16 PM To: hungpham2511/toppra Cc: Subscribed Subject: [hungpham2511/toppra] python requirements boken in latest 16.04 updates (#53)

As of 5/17/2019, Ubuntu apt updates to python-apt packages have broken the toppra requirements install. numpy is no longer supported with python2, so moving from requirements.txt to requirements3.txt got me through the requirements install but then building toppra yells about cython.

You are using pip version 8.1.1, however version 19.1.1 is available. You should consider upgrading via the 'pip install --upgrade pip' command. Traceback (most recent call last): File "setup.py", line 2, in from Cython.Distutils import build_ext ImportError: No module named Cython.Distutils Traceback (most recent call last): File "setup.py", line 2, in from Cython.Distutils import build_ext ImportError: No module named Cython.Distutils make[2]: [CMakeFiles/setup] Error 1 make[1]: [CMakeFiles/setup.dir/all] Error 2 make: *** [all] Error 2

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/hungpham2511/toppra/issues/53?email_source=notifications&email_token=ACKD6TAH7TBZ2ZXP3QFOHP3PWANVZA5CNFSM4HN2QHX2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GURHXGA, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACKD6TAFUGWMDGSH3C74MQTPWANVZANCNFSM4HN2QHXQ.

pbeeson commented 5 years ago

It does work for python3. I got that working eventually.

But I can’t personally pip install toppra because your latest changes break some of my benchmarks tests. Changes you made prior to the auto scaling feature are actually more stable than later versions in my testing. That’s another issue though.