astropy / pyregion

ds9 region parser for python
https://pyregion.readthedocs.io
MIT License
39 stars 38 forks source link

"astropy-helpers==0.4.8" required? #125

Closed matteobachetti closed 6 years ago

matteobachetti commented 6 years ago

Hi,

I'm having a problem installing pyregion lately. You can see an example here: https://travis-ci.org/matteobachetti/srt-single-dish-tools/jobs/326431552

Apparently, installing it through pip requires astropy-helpers 0.4.8, while the version I install is more recent. Is it really a necessity?

Thanks in advance for your help.

Matteo

cdeil commented 6 years ago

@matteobachetti - pyregion bundles astropy-helpers like many other Astropy affiliated packages. It should not require an external astropy-helpers, and certainly not have the version for that pinned to 0.4.8.

Do you still see the issue? Do you have a way to try and figure out what is / was happening?

I just quickly tried this with Python 3.6 and it worked as expected:

$ python -m venv temp
$ temp/bin/pip install pyregion
Collecting pyregion
  Downloading https://files.pythonhosted.org/packages/9d/e9/739397a25a6197f9af396e8ab249ea04d3cc2eb32e96870e2aafcfccf7f4/pyregion-2.0.tar.gz (1.1MB)
    100% |████████████████████████████████| 1.1MB 1.0MB/s 
Collecting pyparsing>=2.0 (from pyregion)
  Downloading https://files.pythonhosted.org/packages/6a/8a/718fd7d3458f9fab8e67186b00abdd345b639976bc7fb3ae722e1b026a50/pyparsing-2.2.0-py2.py3-none-any.whl (56kB)
    100% |████████████████████████████████| 61kB 3.8MB/s 
Collecting numpy (from pyregion)
  Downloading https://files.pythonhosted.org/packages/8e/75/7a8b7e3c073562563473f2a61bd53e75d0a1f5e2047e576ee61d44113c22/numpy-1.14.3-cp36-cp36m-macosx_10_6_intel.macosx_10_9_intel.macosx_10_9_x86_64.macosx_10_10_intel.macosx_10_10_x86_64.whl (4.7MB)
    100% |████████████████████████████████| 4.7MB 288kB/s 
Collecting Cython (from pyregion)
  Downloading https://files.pythonhosted.org/packages/1c/97/f96d3ad8f5365328c0d02090e95e2fcd499825d53947e88ce8e02104507d/Cython-0.28.2-cp36-cp36m-macosx_10_6_intel.macosx_10_9_intel.macosx_10_9_x86_64.macosx_10_10_intel.macosx_10_10_x86_64.whl (5.2MB)
    100% |████████████████████████████████| 5.2MB 265kB/s 
Collecting astropy>=1.0 (from pyregion)
  Downloading https://files.pythonhosted.org/packages/2d/61/84dc098ace93f0f9c9740d3d64031590b2243435b4b828dc1abd14b1faa2/astropy-3.0.2-cp36-cp36m-macosx_10_6_intel.macosx_10_9_intel.macosx_10_9_x86_64.macosx_10_10_intel.macosx_10_10_x86_64.whl (7.5MB)
    100% |████████████████████████████████| 7.5MB 185kB/s 
Installing collected packages: pyparsing, numpy, Cython, astropy, pyregion
  Running setup.py install for pyregion ... done
Successfully installed Cython-0.28.2 astropy-3.0.2 numpy-1.14.3 pyparsing-2.2.0 pyregion-2.0
bsipocz commented 6 years ago

@matteobachetti - Please reopen the issue in ci-helpers, I suspect it's a bug in there somewhere.

bsipocz commented 6 years ago

Oh, one easy thing to try is to add CONDA_CHANNELS=astropy, maybe that would solve the issue (but still, this requirement should not exist.

cdeil commented 6 years ago

I'm closing this issue here. @matteobachetti - If you still see it, please leave another comment or file an issue in the ci-helpers repo as suggested by @bsipocz .

matteobachetti commented 6 years ago

Ok! Thanks @cdeil @bsipocz

On Thu, May 3, 2018 at 8:27 AM Christoph Deil notifications@github.com wrote:

Closed #125 https://github.com/astropy/pyregion/issues/125.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/astropy/pyregion/issues/125#event-1606972437, or mute the thread https://github.com/notifications/unsubscribe-auth/AG22rXqv6Ug-OrdimowcG4Ute7ZSPIQwks5tuqNsgaJpZM4RXrfA .

--


Matteo Bachetti

www.matteobachetti.it

"I would love to change the world, but they won't give me the source code"