parent 0883e92

author Adam Ginsburg (keflavich) <[email protected]> 1660230509 -0400
committer Adam Ginsburg (keflavich) <[email protected]> 1661190088 -0400

attempt to fix broken downloader

account for length-1 lists

account for length-1 lists

fix logic: the number of uids doesn't matter ,the number of files does

the total size doesn't need to be filtered out cleverly, since things
that aren't files don't have size

Author: keflavich

astroquery/alma/core.py

@@ -828,7 +828,15 @@ def retrieve_data_from_uid(self, uids, *, cache=True):
             raise TypeError("Datasets must be given as a list of strings.")
 
         files = self.get_data_info(uids)
-        file_urls = files['access_url']
+        # filter out blank access URLs
+        # it is possible for there to be length-1 lists
+        if len(files) == 1:
+            file_urls = files['access_url']
+            if isinstance(file_urls, str) and file_urls == '':
+                raise ValueError(f"Cannot download uid {uid} because it has no file")
+        else:
+            file_urls = [url for url in files['access_url'] if url]
+
         totalsize = files['content_length'].sum()*u.B
 
         # each_size, totalsize = self.data_size(files)