HERA-Team / aipy

Astronomical Interferometry in PYthon (AIPY)
http://pypi.python.org/pypi/aipy
Other
43 stars 29 forks source link

21cmSense: a problem with aipy downloading and the n.round() function producing non integers #45

Closed binnietom closed 6 years ago

binnietom commented 6 years ago

When trying to run 21cmSense I was getting errors of the form;

IOError: [Errno 2] No such file or directory: '/usr/local/lib/python2.7/dist-packages/aipy/_src/helm_pos.txt'

IOError: [Errno 2] No such file or directory: '/usr/local/lib/python2.7/dist-packages/aipy/_src/helm_fit.txt’

-These can be solved by manually downloading the files into the desired location;

and

''Traceback (most recent call last): File "mk_array_file.py", line 98, in uvsum,quadsum = n.zeros((dim,dim)), n.zeros((dim,dim)) #quadsum adds all non-instantaneously-redundant baselines incoherently TypeError: only integer scalar arrays can be converted to a scalar index’'

which can be solved within "mk_array_file.py" by adding dim = int(dim) to the line before line 98. Also a similar thing needs to be done on line 32 i.e. changing: round(ycen) -- to --> int(round(ycen)) etc.

It then works as expected.

steven-murray commented 6 years ago

I can second both of these issues. I fixed the latter and have create a pull request on jpober/21cmSense. The former issue seems like an oversight in the setup script of aipy. Would be good if it was fixed!

pkgw commented 6 years ago

I had thought that both parts of this were due to 21cmsense being a bit out of date, but it turns out there was a problem in aipy that could be improved. See #49.