tart-telescope / tart2ms

Convert TART data to measurement set format
GNU General Public License v3.0
1 stars 1 forks source link

Ditch Astropy for Skyfield #53

Open bennahugo opened 1 year ago

bennahugo commented 1 year ago

According to testing @jskenyon has been doing on unrelated work the equatorial to local transformations in Skyfield is at least an order of magnitude faster. Curently this is one of the remaining bottlenecks in selecting sources (working out their az el etc. -- this is slower than the actual predict which is done via JIT code).

I feel strongly about ditching astropy so that we can make the codebase as fast as possible to stitch together many days of observation for scientific analysis and classification of the system SEFD once we can pick up astronomical sources like fornax.

tmolteno commented 1 year ago

This would depend a bit on what dependencies skyfield brings in. If it's numba, then we probably can't because of Issue #52