thrustcurve-db - npm Package Compare versions

Comparing version 0.3.13 to 0.3.14

package.json

 {
   "name": "thrustcurve-db",
-  "version": "0.3.13",
+  "version": "0.3.14",
   "type": "module",
@@ -5,0 +5,0 @@ "description": "ThrustCurve.org model rocket motor and thrust curve data as a single JSON file",

util.js

 /**
  * Parse a thrustcurve `delays` value to determine all possible motor delay
- * configurations.
+ * configurations.  This is a "loose" parser (will accept some atypical values),
+ * but delay strings are generally expected to be formatted as follows:
  *
+ * - Delay strings are comma-separated list of values and/or ranges.
+ * - Values are non-zero integers, or an Aerotech letter designation of "S",
+ *   "M", "L", or "X"
+ * - Ranges are the numeric min and max value (inclusive), separated by a dash("-")
+ *
+ * Examples:
+ *
+ *     "4,6,9" = 4, 6, or 9 second delays
+ *      "5-13" = all delays from 5 to 13 seconds (inclusive)
+ * "5-9,11-14" = all delays from 5 to 14 seconds, except 10
+ *
  * @param {String} delays
@@ -40,7 +52,11 @@ * @return {Object} with the following properties:
 
-      // Aerotech letter-delays.  The Aerotech delay drilling tool can remove up
-      // to 8 seconds of delay in 2-second increments.  Aerotech warns against
-      // delays < 6 seconds for in DMS drill tool instructions. And Sirius
-      // Rocketry warns against delays < 4 seconds on their product page for
-      // the RMS drill tool.
+      // Aerotech letter-delays.  The Aerotech delay drill tools can remove up
+      // to 8 seconds of delay in 2-second increments. There are recommended
+      // minimum delay times for the tools, however.  Aerotech says to not set
+      // delays of less than 6 seconds with the DMS tool, while Sirius
+      // Rocketery says no less than 4 seconds for the RMS tool.  We don't
+      // have access to the motor case info here to determine which tool is
+      // involved, but 207 of the 211 AeroTech motors in the TC db are RMS
+      // motors at the time I wrote this, so I'm enforcing a 4 second minimum
+      // here.
       case v === 'S': [4, 6].forEach(times.add, times); continue;
@@ -71,5 +87,10 @@ case v === 'M': [4, 6, 8, 10].forEach(times.add, times); continue;
 
+/**
+ * Convert a `delays` structure returned from parseDelays() into a delays
+ * string.
+ */
 export function unparseDelays(parsed) {
   const times = [...parsed.times].sort((a, b) => a - b);
 
+  /*
   // Tack on an ending value (to flush last value in range aggregator loop,
@@ -103,2 +124,6 @@ // below)
   return vals.join(',');
+  */
+
+  if (parsed.plugged) times.push('P');
+  return times.join(',');
 }

No alert changes

Improved metrics

Total package byte prevSize

1750148

increased by0.12%

Lines of code

121034

increased by0.02%

No dependency changes