Addressing review

Author: bsipocz

astroquery/neodys/core.py

@@ -10,7 +10,7 @@ class NEODySClass(BaseQuery):
     NEODYS_URL = conf.server
     TIMEOUT = conf.timeout
 
-    def query_object(self, object_id, orbital_element_type="eq", epoch_near_present=0):
+    def query_object(self, object_id, *, orbital_element_type="eq", epoch_near_present=0):
         """
         Parameters
         ----------

docs/neodys/neodys.rst

@@ -1,5 +1,3 @@
-.. doctest-skip-all
-
 .. _astroquery.neodys:
 
 ************************************
@@ -16,23 +14,24 @@ matrix are given in dictionary form. The results can be given in either
 Keplerian or Equinoctial. The only required argument is the identifier of the
 object to be retrieved. Here is a basic example:
 
-.. code-block:: python
+.. doctest-remote-data::
 
     >>> from astroquery.neodys import NEODyS
-    >>> res = neodys.core.NEODyS.query_object("1982YA")
-    >>> print(res)
-    {'Equinoctial State Vector': [3.643505823124891, 0.555753608185406,
-    0.413673328231395, -0.316412712941692, -0.005264652421941, 48.553922087068],
-    'Mean Julian Date': ['55417.557065659', 'TDT'], 'Magnitude': [17.798, 0.15],
-    'Covariance Matrix': [6.737866678261391e-15, 3.835925316932412e-14,
-    -6.640239789108145e-14, -1.439007899529439e-15, -4.938922190348063e-15,
-    4.994817272482335e-12, 2.656750560890953e-13, -4.11492595235993e-13,
-    5.390531237956025e-14, -8.210458555809306e-14, 3.034284768527233e-11,
-    6.987414264633028e-13, -3.461847188742589e-14, 8.072873747426578e-14,
-    -5.283334219581364e-11, 3.441265766874457e-13, -1.712409537284e-13,
-    6.293229007026724e-13, 1.418015029378066e-13, -3.898114012957574e-12,
-    4.0664813765478e-09], 'Keplerian Correlation Matrix': [], 'URL':
-    'https://newton.spacedys.com/~neodys2/epoch//1982YA.eq0'}
+    >>> result = NEODyS.query_object("1982YA")
+    >>> print(result)
+    {'Equinoctial State Vector': [3.6501814973859177, 0.55510840531515, 0.412442615432626, -0.316403600209491, -0.005615261288851, 7.4974723045741],
+    'Mean Julian Date': ['57672.370442276', 'TDT'],
+    'Magnitude': [17.478, 0.15],
+    'Covariance Matrix': [2.983481961445874e-14, 1.319692615237118e-13, -3.466482497808619e-13,
+                          -5.179294501167426e-15, 4.40306644125416e-15, 2.841155344781695e-11,
+                          7.248099966950092e-13, -1.55363923471562e-12, -4.018099391315557e-14,
+                          -1.400231787484278e-14, 1.210874090734312e-10, 4.091233073364273e-12,
+                          4.62498475268648e-15, -1.406987981569261e-14, -3.37147537859451e-10,
+                          6.527443331995206e-13, -4.021096836415107e-13, -3.876453730470366e-12,
+                          3.584924689571663e-13, 8.057672786144392e-12, 2.841126641896583e-08],
+    'Keplerian Correlation Matrix': [],
+    'URL': 'https://newton.spacedys.com/~neodys2/epoch//1982YA.eq0'}
+
 
 More detailed parameters
 ------------------------
@@ -44,28 +43,33 @@ and 'eq' for Equinoctial. The other is ``epoch_near_present``, a flag
 which sets the epoc to near present instead of near middle of the arc.
 Valid values are '0' for near middle of the arc, and '1' for near
 present. The default values are 'eq' and '0'. Note: Keplerian elements
-are given to a lower precision on the NEODyS website. 
+are given to a lower precision on the NEODyS website.
 
 Here's an example with these optional parameters:
 
-.. code-block:: python
+.. doctest-remote-data::
 
     >>> from astroquery.neodys import NEODyS
-    >>> res = neodys.core.NEODyS.query_object(
-        "1982YA", orbital_element_type="ke", epoch_near_present=1)
+    >>> results = NEODyS.query_object("1982YA", orbital_element_type="ke", epoch_near_present=1)
     >>> print(results)
-    {'Keplerian State Vector': [3.648336, 0.691188, 35.123, 268.963, 144.376,
-    170.251], 'Mean Julian Date': ['59200.0000', 'TDT'], 'Magnitude': [17.798,
-    0.15], 'Covariance Matrix': [7.39777353e-15, -9.00996211e-15, -3.12399412e-13,
-    -1.41885172e-12, 8.13796963e-12, -2.98795622e-12, 2.61069828e-14,
-    -2.30621969e-12, -3.23293856e-12, -4.26709227e-12, 2.1454954e-12,
-    3.66367782e-09, 3.29118054e-09, -3.80504189e-09, 4.73514563e-10,
-    4.83809216e-09, -6.57723081e-09, 1.26901103e-09, 1.47852822e-08,
-    -4.09276587e-09, 1.39372059e-09], 'Keplerian Correlation Matrix': [1.0,
-    -0.648327, -0.06000688, -0.23716436, 0.77812746, -0.93054173, 1.0,
-    -0.2358108, -0.2876617, -0.21718953, 0.35568159, 1.0, 0.7817286, -0.51699459,
-    0.2095496, 1.0, -0.77766258, 0.4886979, 1.0, -0.90160069, 1.0], 'URL':
-    'https://newton.spacedys.com/~neodys2/epoch//1982YA.ke1'}
+    {'Keplerian State Vector': [3.64669, 0.691945, 35.165, 268.935, 144.408, 283.498],
+    'Mean Julian Date': ['60000.0000', 'TDT'],
+    'Magnitude': [17.478, 0.15],
+    'Covariance Matrix': [3.92261704e-14, -1.16359258e-13, 3.11585918e-13,
+                          5.00974344e-13, 3.35974943e-11, -6.24946297e-12,
+                          4.44803944e-13, 3.19086108e-12, -3.60667109e-12,
+                          -9.43905769e-11, 6.70717262e-12, 6.92715199e-09,
+                          7.66896347e-09, -7.46695141e-09, 1.41600249e-10,
+                          1.22064798e-08, -1.21318576e-08, 1.39344281e-09,
+                          4.21853471e-08, -7.19870851e-09, 2.62151264e-09],
+    'Keplerian Correlation Matrix': [1.0, -0.88090418, 0.01890221,
+                                     0.02289456, 0.82592026, -0.61628098,
+                                     1.0, 0.0574839, -0.0489471,
+                                     -0.68907027, 0.19641705, 1.0,
+                                     0.83399624, -0.43680254, 0.0332285,
+                                     1.0, -0.53462747, 0.24633037,
+                                     1.0, -0.68453837, 1.0],
+    'URL': 'https://newton.spacedys.com/~neodys2/epoch//1982YA.ke1'}
 
 
 Reference/API

docs/solarsystem/solarsystem.rst

@@ -21,6 +21,7 @@ The currently available service providers and services are:
   mpc/mpc.rst
   pds/pds.rst
 
+
 Reference/API
 =============