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

Ensure point_map is written in IMPLICIT FULLSKY #355

Closed
camposandro opened this issue Sep 24, 2024 · 3 comments
Closed

Ensure point_map is written in IMPLICIT FULLSKY #355

camposandro opened this issue Sep 24, 2024 · 3 comments
Assignees
Milestone

Comments

@camposandro
Copy link
Contributor

The point_map.fits sky map is currently written using HEALPy, in the IMPLICIT FULLSKY NESTED format. We should make sure that, after migrating away from HEALPy, we keep it this way. When it comes to that, we should use the methods to read/write sky maps of the CDS HEALPix Rust library (for which we might need interfaces in python).

@nevencaplar
Copy link
Member

@camposandro Is this something that we want to have in before the do renaming HiPSCaT -> HATS

@camposandro
Copy link
Contributor Author

I think we'll only need to worry about this when migrating away from HEALPy. The renaming shouldn't affect it.

@delucchi-cmu
Copy link
Contributor

I would like to include this in the renaming, as we're re-writing most root-level files, and performing dataset conversions in batch. This may be a simple as ensuring that our arguments are:

            hp.write_map(
                <file name>, <map_fits_image>, dtype=np.int32, nest=True, coord="CEL"
            )

@delucchi-cmu delucchi-cmu added this to the HATS 0.4 milestone Sep 26, 2024
@delucchi-cmu delucchi-cmu self-assigned this Sep 27, 2024
@delucchi-cmu delucchi-cmu removed this from the HATS 0.4 milestone Oct 1, 2024
@nevencaplar nevencaplar added this to the HATS 0.4 milestone Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

3 participants