keflavich / pyradex

Python interface to RADEX
BSD 3-Clause "New" or "Revised" License
18 stars 12 forks source link

OS error #12

Closed toto2pech closed 3 years ago

toto2pech commented 7 years ago

Hi, I followed the instructions to install pyradex. In my .bashrc I have the following: export RADEX_DATAPATH=/Users/toto2pech/CODES/RADEX_PYTHON/pyradex and here are the python etc... versions: python -c "import sys, astropy, numpy; print(sys.version); print(numpy.version,astropy.version)" 2.7.12 (default, Oct 11 2016, 14:42:58) [GCC 4.2.1 Compatible Apple LLVM 7.3.0 (clang-703.0.31)] ('1.11.2', u'1.2')

gfortran --version GNU Fortran (MacPorts gcc5 5.4.0_0) 5.4.0 Copyright (C) 2015 Free Software Foundation, Inc.

OS X El Capitan, version 10.11.6

I do have the following error with the h2co_grids.py script (and for other scripts with the same error) that I don't know how to deal with:

python h2co_grids.py Traceback (most recent call last): File "h2co_grids.py", line 30, in R = pyradex.Radex(species='oh2co-h2', abundance=abundance) File "/Users/toto2pech/CODES/RADEX_PYTHON/pyradex/pyradex/core.py", line 314, in init raise ValueError("Must specify two of column, density, and abundance.") ValueError: Must specify two of column, density, and abundance. Many thanks for your help!

keflavich commented 7 years ago

Your RADEX_DATAPATH should not include your global PATH. Instead of:

export RADEX_DATAPATH=/Users/toto2pech/CODES/RADEX_PYTHON/pyradex:$PATH

you want

export RADEX_DATAPATH=/Users/toto2pech/CODES/RADEX_PYTHON/pyradex
keflavich commented 7 years ago

Yes, that looks like a real error. Would you be interested in submitting a pull request to fix it?

toto2pech commented 7 years ago

On 14 Dec 2016, at 16:38, Adam Ginsburg notifications@github.com wrote:

Yes, that looks like a real error. Would you be interested in submitting a pull request to fix it?

I updated my “request” on the website. Thanks!!

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.