search

desihub / desiconda

Tools to install DESI dependencies from scratch.
BSD 3-Clause "New" or "Revised" License
1 stars 7 forks source link

Add kapteyn package #32

Closed tskisner closed 1 year ago

tskisner commented 5 years ago

This is needed by tilepicker.

Srheft commented 5 years ago

Thanks, Ted; According to my google search (heh), the incompatibility of kapteyn with python 3.7 is an observed/known issue. Hopefully by the time that desiconda would want to upgrade to 3.7, that issue is resolved.

sbailey commented 5 years ago

@srheft in the spirit of limiting our required dependencies, could the functionality of this be handled by astropy functions instead? Specifically what features of kapteyn are needed? It looks like it has some useful tools but only a small contributor base and isn't a broadly used community package, thus I'm hesitant to bring it on as a required dependency for the next 5+ years.

Srheft commented 5 years ago

we can check to see if wcs.Transformation in kapteyn can be replaced by one of the astropy.wcs projections.

weaverba137 commented 1 year ago

Given the dates, I assume that the problem was indeed solved using astropy. Feel free to reopen if not.