make Skyhealpixs a subclass of Skybricks and factor out common code

desihub / desitarget

DESI Targeting

BSD 3-Clause "New" or "Revised" License

18 stars 23 forks source link

Closed dstndstn closed 1 year ago

dstndstn commented 2 years ago

This gives them the same API, without the need to create a skybricks-exist index file for the healpixes.

geordie666 commented 2 years ago

Just a reminder: don't forget to update the doc/changes.rst file to describe this PR before merging.

geordie666 commented 2 years ago

@dstndstn: I have no objection to your merging this (after resolving conflicts). Are you waiting for a review?