-
-
Notifications
You must be signed in to change notification settings - Fork 427
EUCLID: new x-match method #3386
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
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -23,6 +23,7 @@ | |||||
| from astroquery import log | ||||||
| from astroquery.utils import commons | ||||||
| from astroquery.utils.tap import TapPlus | ||||||
| from astroquery.utils.tap import taputils | ||||||
| from . import conf | ||||||
|
|
||||||
|
|
||||||
|
|
@@ -116,6 +117,156 @@ def __init__(self, *, environment='PDR', tap_plus_conn_handler=None, datalink_ha | |||||
| if show_server_messages: | ||||||
| self.get_status_messages() | ||||||
|
|
||||||
| def cross_match_basic(self, *, table_a_full_qualified_name, table_a_column_ra, table_a_column_dec, | ||||||
| table_b_full_qualified_name='', table_b_column_ra='', table_b_column_dec='', | ||||||
|
||||||
| results_name=None, | ||||||
| radius=1.0, background=False, verbose=False): | ||||||
| """Performs a positional cross-match between the specified tables. | ||||||
|
|
||||||
| This method carries out the following steps in one step: | ||||||
|
|
||||||
| 1. updates the user table metadata to flag the positional RA/Dec columns; | ||||||
| 2. launches a positional cross-match as an asynchronous query; | ||||||
| 3. returns all the columns from both tables plus the angular distance (deg) for the cross-matched sources. | ||||||
|
|
||||||
| The result is a join table with the identifies of both tables and the distance (degrees), that is returned | ||||||
| without metadata units. If desired, units can be added using the Units package of Astropy as follows: | ||||||
| results[‘separation’].unit = u.degree. To speed up the cross-match, pass the biggest table to the | ||||||
|
||||||
| results[‘separation’].unit = u.degree. To speed up the cross-match, pass the biggest table to the | |
| ``results[‘separation’].unit = u.degree``. To speed up the cross-match, pass the biggest table to the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes committed to the branch.
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use a more specific, expected exception type here as not all the problems with be due to not having the table in the archive.
(I see this problem exists in the gaia module, too)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The aim is to catch any exception and not to continue with the process.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeap, but then just let them raise as they come as the new exception may not be correct.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The exception is propagated. Changes committed to the branch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be just named
cross_match, or are there plans to have a more generic one like we have for gaia?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the idea is to follow the same names as in gaia. We are not 100 % that the "other" x-match will be implemented.