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

Source names query breaks when not all sources are found #484

Open
ddobie opened this issue Aug 29, 2023 · 4 comments
Open

Source names query breaks when not all sources are found #484

ddobie opened this issue Aug 29, 2023 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@ddobie
Copy link
Collaborator

ddobie commented Aug 29, 2023

Kindly pointed out by @Rachel-Berdugo.

I suspect we just need to drop sources earlier, but will look into this post-workshop.

@ddobie ddobie added the bug Something isn't working label Aug 29, 2023
@ddobie ddobie self-assigned this Aug 29, 2023
@ddobie
Copy link
Collaborator Author

ddobie commented Sep 1, 2023

This issue has two causes.

  1. We don't handle the simbad query properly and just assume that each requested name will be successfully resolved, and only resolved into one source. Anything other than a one-to-one match results in things breaking.
  2. Rachel's specific problem was being caused by one of her objects having the name "AT". Querying simbad for this name (along with at least one other real name) results in it returning every object with an "AT" prefix, of which there is ~3000.

I think the solution is to just drop all sources that don't have a TYPED_ID.

@ddobie
Copy link
Collaborator Author

ddobie commented Sep 1, 2023

As an aside, there's a lot of claptrap that we have in query.py from before we were able to extract the originally queried name. Most of that can be removed now that TYPED_ID is returned.

@ddobie
Copy link
Collaborator Author

ddobie commented Sep 1, 2023

Also weirdly enough, this bug only occurs when you query "AT" along with at least one other name. Querying "AT" by itself results in the expected behaviour (a single source with no match).

@ddobie
Copy link
Collaborator Author

ddobie commented Sep 1, 2023

The same behaviour occurs for "GRB", but not for "SN" or "M". Querying "NGC" results in a timeout.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant