NASA-NAVO / navo-workshop

Tutorial notebooks for how to use PyVO to access NASA and other data in Python.
https://NASA-NAVO.github.io/navo-workshop
BSD 3-Clause "New" or "Revised" License
41 stars 26 forks source link

Update installation files and instructions #119

Closed stargaser closed 1 year ago

stargaser commented 1 year ago

Fixes #117

stargaser commented 1 year ago

Just updating my local miniconda didn't give me the "incompatible architecture" problem. I'm assuming that issue was on an Intel Mac. Was it with just the latest Python 3.10 version or was it also seen with a fresh install of the latest Python 3.9 version?

Anastasia encountered this on her Intel Mac with the latest version which must be Python 3.10. The second newest worked which must be the Python 3.9 version.

bsipocz commented 1 year ago

latest version which must be Python 3.10

not python 3.11?

bsipocz commented 1 year ago

I would still suggest trying adding the different solver, that shouldn't add confusion, but would provide a much faster experience.

tomdonaldson commented 1 year ago

I would still suggest trying adding the different solver, that shouldn't add confusion, but would provide a much faster experience.

I won't object to that, though I think it obligates some of us to do a little testing of the new instructions. Should we do that here, or should I merge this and look for a proposed change in another PR?

tomdonaldson commented 1 year ago

FWIW, I haven't been able to reproduce the miniconda error. Obviously there could be a lot of variables, but both of these downloaded (with correct SHA256 hash), installed and worked on my Intel Mac (MacOS 12.6.6 Monterey):

Miniconda3-py310_23.3.1-0-MacOSX-x86_64.pkg
Miniconda3-py310_23.3.1-0-MacOSX-x86_64.sh

This does make me wonder if the problem might not be very widespread.

I'll go ahead and merge this. Any new instructions can come in a new PR.

stargaser commented 1 year ago

Okay, I've updated to remove mamba and add in the libmamba solver. See if this makes sense.

stargaser commented 1 year ago

Oops, never mind, it's been merged, I missed that. Let's pu the change in another PR.