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

multiprocessing library is a dependency when installing with pip #2

Open
lucydot opened this issue Mar 23, 2021 · 2 comments
Open

multiprocessing library is a dependency when installing with pip #2

lucydot opened this issue Mar 23, 2021 · 2 comments
Assignees

Comments

@lucydot
Copy link

lucydot commented Mar 23, 2021

Describe the bug
Thanks for the excellent looking package and clear documentation. I tried to install pieface using conda+pip. There was an error and install was unsuccessful. It is the error reported here: https://stackoverflow.com/questions/42568638/python-multiprocessing-installation-command-python-setup-py-egg-info-failed-w. This is unexpected as I can see that you have removed the multiprocessing library as a dependency in setup.py. Perhaps this is not reflected in the version you have uploaded to PyPI? pip is collecting pieface 1.1.0.

To Reproduce

conda create --name pieface
conda activate pieface
conda install pip
pip install pieface

Expected behavior
Succesful installation

Screenshots and Files
N/A

Operating system (please complete the following information):

  • OS: Mac
  • Python version: Python3.9.2
  • PIEFACE version: 1.1.0

Additional context
The workaround that worked for me is to install the dependencies in setup.py separately and then pip install --no-dependencies pieface. I'm just flagging this up as a FYI.

@lucydot
Copy link
Author

lucydot commented Mar 23, 2021

Hello,

Just following up on this. Although it installed correctly with the workaround above, I still had problems as I was using Python 3. I specified Python 2.7 in my conda environment and everything worked as expected (multiprocessing was installed as a dependency without problems).

conda create --name pieface python=2.7
conda activate pieface
conda install pip
pip install pieface

As most people / projects are on Python3 as default now it might be worth flagging up in the README that Python2 is the supported language.

Thanks again for the code, I'm just waiting for it to churn through some cifs at the moment -

Lucy

@jcumby
Copy link
Owner

jcumby commented Mar 29, 2021

Thanks for the bug report @lucydot! You're spot on; I removed the dependency on multiprocessing in the development version, but had not released a newer version to PyPI.

I'm a long way behind in moving to Python3, but it is on the to-do list. My plan is also to distribute using conda in the future (rather than the windows installer or Pip); my hope is I might find time this summer to work on all this!

@jcumby jcumby self-assigned this Mar 29, 2021
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

2 participants