Refactor: Made ipac/irsa/irsa_dust kwargs keyword only

Author: nkphysics

astroquery/ipac/irsa/irsa_dust/core.py

@@ -48,7 +48,7 @@ class IrsaDustClass(BaseQuery):
         '100um': 'e'
     }
 
-    def get_images(self, coordinate, radius=None,
+    def get_images(self, coordinate, *, radius=None,
                    image_type=None, timeout=TIMEOUT, get_query_payload=False,
                    show_progress=True):
         """
@@ -91,7 +91,7 @@ def get_images(self, coordinate, radius=None,
             get_query_payload=get_query_payload, show_progress=show_progress)
         return [obj.get_fits() for obj in readable_objs]
 
-    def get_images_async(self, coordinate, radius=None, image_type=None,
+    def get_images_async(self, coordinate, *, radius=None, image_type=None,
                          timeout=TIMEOUT, get_query_payload=False,
                          show_progress=True):
         """
@@ -141,7 +141,7 @@ def get_images_async(self, coordinate, radius=None, image_type=None,
                                       show_progress=show_progress)
                 for U in image_urls]
 
-    def get_image_list(self, coordinate, radius=None, image_type=None,
+    def get_image_list(self, coordinate, *, radius=None, image_type=None,
                        timeout=TIMEOUT):
         """
         Query function that performs coordinate-based query and returns a list
@@ -182,7 +182,7 @@ def get_image_list(self, coordinate, radius=None, image_type=None,
                                  timeout=timeout)
         return self.extract_image_urls(response.text, image_type=image_type)
 
-    def get_extinction_table(self, coordinate, radius=None, timeout=TIMEOUT,
+    def get_extinction_table(self, coordinate, *, radius=None, timeout=TIMEOUT,
                              show_progress=True):
         """
         Query function that fetches the extinction table from the query
@@ -220,7 +220,7 @@ def get_extinction_table(self, coordinate, radius=None, timeout=TIMEOUT,
                 column.unit = str(column.unit)[:-1]
         return table
 
-    def get_extinction_table_async(self, coordinate, radius=None,
+    def get_extinction_table_async(self, coordinate, *, radius=None,
                                    timeout=TIMEOUT, show_progress=True):
         """
         A query function similar to
@@ -253,11 +253,11 @@ def get_extinction_table_async(self, coordinate, radius=None,
         response = self._request("POST", url, data=request_payload,
                                  timeout=timeout)
         xml_tree = utils.xml(response.text)
-        result = SingleDustResult(xml_tree, coordinate)
+        result = SingleDustResult(xml_tree, query_loc=coordinate)
         return commons.FileContainer(result.ext_detail_table(),
                                      show_progress=show_progress)
 
-    def get_query_table(self, coordinate, radius=None,
+    def get_query_table(self, coordinate, *, radius=None,
                         section=None, timeout=TIMEOUT, url=DUST_SERVICE_URL):
         """
         Create and return an `~astropy.table.Table` representing the query
@@ -299,7 +299,7 @@ def get_query_table(self, coordinate, radius=None,
         response = self._request("POST", url, data=request_payload,
                                  timeout=timeout)
         xml_tree = utils.xml(response.text)
-        result = SingleDustResult(xml_tree, coordinate)
+        result = SingleDustResult(xml_tree, query_loc=coordinate)
         if section is None or section in ["location", "loc", "l"]:
             return result.table(section=section)
         try:
@@ -310,7 +310,7 @@ def get_query_table(self, coordinate, radius=None,
                    'ebv, temperature, location or 100um.')
             raise ValueError(msg)
 
-    def _args_to_payload(self, coordinate, radius=None):
+    def _args_to_payload(self, coordinate, *, radius=None):
         """
         Accepts the query parameters and returns a dictionary
         suitable to be sent as data via a HTTP POST request.
@@ -357,7 +357,7 @@ def _args_to_payload(self, coordinate, radius=None):
             payload["regSize"] = reg_size
         return payload
 
-    def extract_image_urls(self, raw_xml, image_type=None):
+    def extract_image_urls(self, raw_xml, *, image_type=None):
         """
         Extracts the image URLs from the query results and
         returns these as a list. If section is missing or
@@ -414,7 +414,7 @@ class SingleDustResult:
     initial response. Not intended to be instantiated by the end user.
     """
 
-    def __init__(self, xml_tree, query_loc=None):
+    def __init__(self, xml_tree, *, query_loc=None):
         """
         Parameters
         ----------
@@ -450,7 +450,7 @@ def xml(self):
         """Return the raw xml underlying this SingleDustResult."""
         return self._xml
 
-    def table(self, section=None):
+    def table(self, *, section=None):
         """
         Create and return a `~astropy.table.Table` representing the query
         response.
@@ -467,7 +467,7 @@ def table(self, section=None):
         else:
             return self._table(code)
 
-    def values(self, section=None):
+    def values(self, *, section=None):
         """
         Return the data values contained in the query response,
         i.e. the list of values corresponding to a row in the result table.
@@ -733,7 +733,7 @@ class NumberNode(BaseDustNode):
     number.
     """
 
-    def __init__(self, xml_node, col_name, units=None):
+    def __init__(self, xml_node, col_name, *, units=None):
         """
         Parameters
         ----------
@@ -881,7 +881,7 @@ def __init__(self, xml_root):
         # Create the section's DustNodes
         self._dust_nodes = [CoordNode(
             xml_nodes[OBJ_NAME], col_names=["RA", "Dec", "coord sys"]),
-            NumberNode(xml_nodes[REG_SIZE], REG_SIZE, u.deg)]
+            NumberNode(xml_nodes[REG_SIZE], REG_SIZE, units=u.deg)]
 
         self.create_columns()
 
@@ -899,7 +899,7 @@ class StatsSection(BaseResultSection):
     section.
     """
 
-    def __init__(self, xml_root, col_prefix, suffix=""):
+    def __init__(self, xml_root, col_prefix, *, suffix=""):
         """
         Parameters
         ----------
@@ -984,8 +984,8 @@ def __init__(self, xml_root):
 
         # Create statistics subsections
         self._stats_sandf = StatsSection(xml_nodes[STATISTICS],
-                                         "ext SandF", "SandF")
-        self._stats_sfd = StatsSection(xml_nodes[STATISTICS], "ext SFD", "SFD")
+                                         "ext SandF", suffix="SandF")
+        self._stats_sfd = StatsSection(xml_nodes[STATISTICS], "ext SFD", suffix="SFD")
 
         self.create_columns()