Skip to content

Commit

Permalink
Merge pull request #279 from DavidT3/bug/XMMTableChanged
Browse files Browse the repository at this point in the history
Jessica (@jessicapilling) noticed the online table for XMM evidently …
  • Loading branch information
DavidT3 authored May 1, 2024
2 parents 64c3570 + 011c08c commit 85fbc87
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions daxa/mission/xmm.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# This code is a part of the Democratising Archival X-ray Astronomy (DAXA) module.
# Last modified by David J Turner (turne540@msu.edu) 24/04/2024, 13:58. Copyright (c) The Contributors
# Last modified by David J Turner (turne540@msu.edu) 01/05/2024, 10:25. Copyright (c) The Contributors
import os.path
import tarfile
from datetime import datetime
Expand Down Expand Up @@ -185,7 +185,7 @@ def _fetch_obs_info(self):
# I'll explain in a second
count_tab = AQXMMNewton.query_xsa_tap('select count(observation_id) from xsa.v_all_observations')
# Then I round up to the nearest 1000, probably unnecessary but oh well
num_obs = np.ceil(count_tab['count'].tolist()[0] / 1000).astype(int) * 1000
num_obs = np.ceil(count_tab['COUNT'].tolist()[0] / 1000).astype(int) * 1000
# Now I have to be a bit cheesy - If I used select * (which is what I would normally do in an SQL-derived
# language to grab every row) it actually only returns the top 2000. I think that * is replaced with TOP 2000
# before the query is sent to the server. However if I specify a TOP N, where N is greater than 2000, then it
Expand Down

0 comments on commit 85fbc87

Please sign in to comment.