-
Notifications
You must be signed in to change notification settings - Fork 16
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
v2: region, bins, healpix, ztffields #52
Comments
@MickaelRigault Yup! We have some example interactions coded up here: https://github.com/skyportal/skyportal/blob/main/skyportal/handlers/api/instrument.py#L519 |
This seems close to shapely: I'll have a closer look. |
@MickaelRigault Indeed. To me, the important thing is interactions with the healpix stuff. |
do you have an example of that too (never used it before) |
@MickaelRigault healpy has nice utility functions for extracting indices from shapes: https://healpy.readthedocs.io/en/latest/generated/healpy.query_polygon.html |
@mcoughlin I currently call it skysurvey, we can easily merge it with simsurvey if we want to keep the simsurvey "brand". Moving the discussion above there. You will see basic implementation started. |
Really interesting @MickaelRigault! Personally I think that yes this should become the new simsurvey. Presumably the tag should be a 1.X release to indicate potentially breaking changes once it is ready. |
Fine by me |
Hi @mcoughlin you mentioned you wanted to use astropy regions,
this is what you had in mind right ?
https://astropy-regions.readthedocs.io/en/stable/
The text was updated successfully, but these errors were encountered: