-
Notifications
You must be signed in to change notification settings - Fork 25
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
Remove dependency on SciPy #313
Comments
Also remove Pandas since it's not used anywhere either. |
Also remove sympy |
also remove nose |
Remove "rplidar-roboticia" from Body's setup.py, but keep it in Tool's setup.py |
remove snakeviz |
remove pyusb from Body's setup.py, but keep it in Tool's setup.py |
Remove SpeechRecognition |
Remove pixel-ring |
Remove cma |
remove scikit-image |
remove open3d |
Remove urdf_parser_py from Body's setup.py (it is not in Tool's setup.py) |
Remove jsonschema, qtconsole, and jupyter. I believe we had these pinned for a bug related to Python2. Line 27 in a97f03f
|
Remove llvmlite. I believe we had it pinned for a Numba bug related to Python2. Line 28 in a97f03f
|
Remove terminado. Line 29 in a97f03f
|
Remove condition on Dynamixel SDK. Now that we only use Python3, we don't need ship versions of Stretch Body that support Py2. https://github.com/hello-robot/stretch_body/blob/master/body/setup.py#L30C37-L30C108 |
Remove condition on PyYaml. The latest is 6.0.1. https://github.com/hello-robot/stretch_body/blob/master/body/setup.py#L31C29-L31C68 |
Remove numpy-stl from Body's setup.py |
Remove playsound from Body's setup.py |
See if we can remove the pins on pyglet and trimesh in the Tool's setup.py. I think since we started using urchin instead of urdfpy, we no longer need those pins. https://github.com/hello-robot/stretch_body/blob/master/tools/setup.py#L32C23-L32C85 |
See if we can add a dependency on pyaudio here, and remove it from Stretch Install: |
See if we can add a dep on pyserial here, and remove it from Stretch Install: |
Both
setup.py
include SciPy as a dependency, when it's not used anywhere in the code. Therefore, it's safe to remove the dependency.The text was updated successfully, but these errors were encountered: