Merge pull request #459 from fchapoton/typos_and_other_details

a few typos and minor details

Author: ChristopherRabotin

anise-py/tests/test_almanac.py

@@ -197,7 +197,7 @@ def test_convert_tpc():
     almanac = new_meta.process()
 
     earth_j2k = almanac.frame_info(Frames.EARTH_J2000)
-    assert earth_j2k.mu_km3_s2 != None
+    assert earth_j2k.mu_km3_s2 is not None
     almanac.describe()
 
 

anise-py/tutorials/Tutorial 05 - Using frame kernels and text planetary kernels.ipynb

@@ -32,7 +32,7 @@
                 "\n",
                 "### Version compatibility\n",
                 "\n",
-                "ANISE guarantees to _not_ change the structure of these kernels in between patch versions (e.g. version `0.3.0` and version `0.3.99` are garanteed to have the compatible kernels). However, until version `1.0.0`, the structure _may_ change and if so, the updated version of the default PCA and EPA files will be added to the Nyx Space cloud.\n",
+                "ANISE guarantees to _not_ change the structure of these kernels in between patch versions (e.g. version `0.3.0` and version `0.3.99` are guaranteed to have the compatible kernels). However, until version `1.0.0`, the structure _may_ change and if so, the updated version of the default PCA and EPA files will be added to the Nyx Space cloud.\n",
                 "\n",
                 "Since version `0.1.0`, the structure of the kernels has _not_ changed. However, the ANISE version is encoded at the start of each kernel. This is only used if the data set cannot be properly decoded to inform the user of the expected ANISE version and the one that they're trying to load. In other words, although there is a version `0.3` of the PCK08 and PCK11 kernels, the files used in version `0.1.0` are still compatible."
             ]

anise/src/naif/kpl/parser.rs

@@ -125,7 +125,7 @@ pub fn parse_bytes<R: BufRead, I: KPLItem>(
                 };
                 assignments.push(assignment);
             } else if let Some(mut assignment) = assignments.pop() {
-                // This is a continuation of the previous line, so let's grab the data and append the value we're reding now.
+                // This is a continuation of the previous line, so let's grab the data and append the value we're reading now.
                 // We're adding the full line with all of the extra spaces because the parsing needs those delimiters to not bunch together all of the floats.
                 assignment.value += &line;
                 assignments.push(assignment);

anise/tests/ephemerides/transform.rs

@@ -219,7 +219,7 @@ fn spice_verif_iau_moon(almanac: Almanac) {
     dbg!(rss_pos_km, rss_vel_km_s);
 
     // ANISE uses hifitime which is more precise than SPICE at time computations.
-    // The Moon angular acceleration is expressed in centuries sicne J2000, where Hifitime does not suffer from rounding errors.
+    // The Moon angular acceleration is expressed in centuries since J2000, where Hifitime does not suffer from rounding errors.
     assert!(rss_pos_km < 0.004);
     assert!(rss_vel_km_s < 1e-5);
 }

anise/tests/test_analysis/lro-occultation.py

@@ -28,4 +28,3 @@
     # Compute occultation
     occult = sp.occult(front, fshape, fframe, back, bshape, bframe, abcorr, obsrvr, epoch)
     assert occult == rslt, f"want {rslt} got {occult}"
-

anise/tests/test_analysis/spk_validation_plots.py

@@ -7,10 +7,8 @@
 
 
 def is_on_github_actions():
-  if "CI" not in environ or not environ["CI"] or "GITHUB_RUN_ID" not in environ:
-    return False
-  else:
-    return True
+    return ("CI" in environ and environ["CI"] and "GITHUB_RUN_ID" in environ)
+
 
 if __name__ == '__main__':
 
@@ -36,7 +34,7 @@ def is_on_github_actions():
 
             plt = px.scatter(subset,
                              x='ET Epoch (s)',
-                             y=f'Absolute difference',
+                             y='Absolute difference',
                              color='source frame',
                              title=f"Validation of {name} for {kind}")