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

esa.iso service error #2344

Closed
bsipocz opened this issue Mar 29, 2022 · 2 comments · Fixed by #2351
Closed

esa.iso service error #2344

bsipocz opened this issue Mar 29, 2022 · 2 comments · Fixed by #2351

Comments

@bsipocz
Copy link
Member

bsipocz commented Mar 29, 2022

The download data tests results a 500 service error:

In [1]: from astroquery.esa.iso import ISO

In [2]:         parameters = {'tdt': "40001501",
   ...:                       'level': "DEFAULT_DATA_SET",
   ...:                       'retrieval_type': "OBSERVATION",
   ...:                       'filename': "file",
   ...:                       'verbose': False}
   ...:         ida = ISO()
   ...:         res = ida.download_data(**parameters)
---------------------------------------------------------------------------
HTTPError                                 Traceback (most recent call last)
<ipython-input-2-c44618bb6568> in <module>
      5               'verbose': False}
      6 ida = ISO()
----> 7 res = ida.download_data(**parameters)

~/munka/devel/astroquery/astroquery/esa/iso/core.py in download_data(self, tdt, retrieval_type, filename, verbose, **kwargs)
    124 
    125         response = self._request('GET', link, save=False, cache=True)
--> 126         response.raise_for_status()
    127 
    128         # Get original extension

~/.pyenv/versions/3.9.1/lib/python3.9/site-packages/requests/models.py in raise_for_status(self)
    941 
    942         if http_error_msg:
--> 943             raise HTTPError(http_error_msg, response=self)
    944 
    945     def close(self):

HTTPError: 500 Server Error:  for url: http://nida.esac.esa.int/nida-sl-tap/data?retrieval_type=OBSERVATION&DATA_RETRIEVAL_ORIGIN=astroquery&tdt=40001501&level=DEFAULT_DATA_SET

Copy pasting the URL to the browser ends up with a bit more details, but it needs someone from the server-side to make sense of the trace.

cc @jesusjuansalgado

@jespinosaar
Copy link
Contributor

Hi @bsipocz ,

I am taking care of this issue and I will also be in charge of this module for further updates.

And @jesusjuansalgado thanks for letting us know about it!

@jespinosaar
Copy link
Contributor

I have created this PR to fix this issue:
#2351

@bsipocz bsipocz linked a pull request Mar 29, 2022 that will close this issue
bsipocz added a commit that referenced this issue Mar 29, 2022
…ailing_remote_test

Issue #2344: test fixed, examples in doc tested and updated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants