gbrammer / eazy-py

Pythonic photometric redshift tools based on EAZY
MIT License
37 stars 25 forks source link

Binder Demo: Building wheel for grizli (setup.py) ... error #10

Closed njcleri closed 3 years ago

njcleri commented 3 years ago

@gbrammer I am just starting to work with eazy-py and in the binder demo I keep getting errors on the importing/installing of grizli in the second cell: image followed by a slew of other red error messages.

I have tried on a few different machines to see if that was the issue and am still getting the same result. (As a disclaimer, this is my first time using binder so it could well be that.)

njcleri commented 3 years ago

Update: I had success with this by downloading the demo and running it directly on my machine with the grizli import commented out.

gbrammer commented 3 years ago

Glad you got it to work. It should also work with a “!pip install cython” before the pip install grizli. I’ll update the notebook accordingly.

Thanks Gabe

On 16 Jun 2021, at 20.12, Nikko Cleri @.***> wrote:

 Update: I had success with this by downloading the demo and running it directly on my machine with the grizli import commented out.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

gbrammer commented 3 years ago

My comment above about installing cython is now implemented in the demo notebook (https://github.com/gbrammer/eazy-py/commit/23b53b65e0ce5bb4371e3effe4e3a80b2b364b50), and the binder seems to run. Thanks for pointing out the issue, @njcleri.