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

Add protection to ensure that preComputedMap has the same NSIDE as requested #23

Open
rbiswas4 opened this issue Aug 1, 2016 · 0 comments

Comments

@rbiswas4
Copy link
Owner

rbiswas4 commented Aug 1, 2016

In the method of instantiating HealpixTiles using preComputedMap, it is possible to instantiate HealpixTiles with a NSIDE different from the one used to write preComputedMap. There should be conditionals to ensure that different values result in raising a ValueError. The check could be related to checking the number of distinct ipix values in the preComputedMap. The problem is the number of distinct values may be smaller than what is obtained from nside2npix because for example the pixels associated with the Northern Hemisphere will not be represented. So, this requires some form of additional information being written out to preComputedMap. Thus this also needs some work in OpSimSummary. That is represented in https://github.com/rbiswas4/OpSimSummary/issues/53

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant