Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
binnietom opened this issue Jul 23, 2018 · 2 comments

Comments

@binnietom
Copy link

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
Copy link
Contributor

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 added a commit to pkgw/aipy that referenced this issue Sep 3, 2018
It turns out that package_data is not the way to do this anymore.
Closes HERA-Team#45.
@pkgw
Copy link
Contributor

pkgw commented Sep 3, 2018

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.

@pkgw pkgw closed this as completed in #49 Sep 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants