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

distance calculation in AXS #22

Open
AltieriBruno opened this issue Jul 23, 2020 · 2 comments
Open

distance calculation in AXS #22

AltieriBruno opened this issue Jul 23, 2020 · 2 comments

Comments

@AltieriBruno
Copy link

We have found an issue with the distance computed by calcGnom for catalogue cross-correlation in AXS :

def calcGnom = (r1: Double, r2: Double, d1: Double, d2: Double) => {

Which for very small angles is ~ SQRT( delta(ra)^2 + delta(dec)^2 )
while in the equirectangular approximation it is ~ SQRT( (delta(ra) * cos(dec1+dec2)/2 )^2 + delta(dec)^2 )

The missing factor of the cosine of the declination yields to an overestimated distance away from the equator.

My impression is that the Gnomonic projection is not suitable in astronomy and is more related to geophysics.

This leads to an actual loss of cross-correlated sources at high latitudes.

@NigelHambly
Copy link

Hi Folks: we're extremely interested in using AXS on a code-to-data platform for Gaia science, but see that some issues have been identified with the spherical trig for computing correctly the matches as one moves away from the equator, or across the RA 0h boundary etc. Are there any plans for pulling in the fixes being worked on by @stargaser and making a new release? Let us know if we can help with testing/validating any branches/forks.

@ctslater
Copy link
Member

Hi, yes I'm working on the distance computation issue now. I've also brought the code up to spark 3.0, so we will make a new release based on that once the distances are sorted out. Thanks for the offer to help with testing; I'll post back here once we have a release candidate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants