mechanical-tolerance-calculator 1.0.5 → 1.0.7

package/index.js

@@ -96,17 +96,34 @@ function returnTolerancesFor(executableMaterialType, spec = "") {
   };
 }
 
-function parseNominalFromMeasurement(measurement, materialType) {
+function parseNominalFromMeasurement(
+  measurement,
+  materialType,
+  THRESHOLD = 0.9
+) {
+  const lowerNominal = Math.floor(measurement);
+
   // For shafts: upper_deviation is 0, so measurement ≤ nominal
   // Therefore, nominal must be ceiling of measurement
   if (materialType === "shafts") {
+    const standardNominal = Math.ceil(measurement); //a standard shaft will always have measurements less than the nominal
+
+    //however, in some cases, we get shafts going beyond the upper deviation
+    //so, we work with a threshold of 0.10 (meaning, a shaft can only go upto 0.10 of it's upper deviation)
+    if (standardNominal - measurement >= THRESHOLD) {
+      return Math.floor(measurement);
+    }
     return Math.ceil(measurement);
   }
 
   // For bores: lower_deviation is 0, so measurement ≥ nominal
   // Therefore, nominal must be floor of measurement
   if (materialType === "housingBores" || materialType === "shellBores") {
-    return Math.floor(measurement);
+    const standardNominal = Math.floor(measurement);
+
+    return measurement - standardNominal >= THRESHOLD
+      ? Math.ceil(measurement)
+      : standardNominal;
   }
 
   // Default: round to nearest
@@ -196,13 +213,13 @@ function processMeasurement(materialType, measurement, tolerances) {
 
   // Check if measurement meets specification
   const meetsSpec = checkMeetsSpecification(measurement, computedBounds);
-  const specMeetingReason = meetsSpec
-    ? `${parseToFixedThreeString(measurement)} falls between ${
-        computedBounds.lowerBound
-      } and ${computedBounds.upperBound}`
-    : `${parseToFixedThreeString(measurement)} doesn't fall between ${
-        computedBounds.lowerBound
-      } and ${computedBounds.upperBound}`;
+
+  const specMeetingReason = generateReasonForSpecs(
+    meetsSpec,
+    measurement,
+    computedBounds.lowerBound,
+    computedBounds.upperBound
+  );
 
   return {
     measurement: parseStringFloat(measurement),
@@ -225,7 +242,7 @@ function processOneMeasurement(materialType, measurement, tolerances) {
   );
   return {
     ...processedMeasurement,
-    meets_IT_tolerance: processedMeasurement.meet_specification,
+    meets_IT_tolerance: processedMeasurement.meets_specification.meetsSpec,
   };
 }
 
@@ -303,21 +320,13 @@ function parseStringFloat(value) {
   // Return 0 if parsing fails (NaN)
   return isNaN(parsed) ? 0 : parsed;
 }
-function processIndividualMeasurement(
-  materialType,
-  measurement,
-  tolerances,
-  meetsIT,
-  ITMeetingReason
-) {
+function processIndividualMeasurement(materialType, measurement, tolerances) {
   const processedMeasurement = processMeasurement(
     materialType,
     measurement,
     tolerances
   );
-  return {
-    ...processedMeasurement,
-  };
+  return processedMeasurement;
 }
 
 function checkMultipleMeasurementsFor(materialType, measurements) {
@@ -333,48 +342,88 @@ function checkMultipleMeasurementsFor(materialType, measurements) {
     largestMeasurement - smallestMeasurement
   );
 
-  const ITs = {
-    IT5: 0,
-    IT6: 0,
-    IT7: 0,
-    IT8: 0,
-    IT9: 0,
-  };
-
+  let mostFarMeasurement = largestMeasurement;
+  const nominals = {};
+  let count = 0;
+  let withInSpecs = [];
   const results = measurements.map((measurement) => {
     const result = processIndividualMeasurement(
       camcoStandardTolerances.type,
       measurement,
       camcoStandardTolerances
     );
-
-    ITs[result.IT_grade]++;
+    withInSpecs.push(result.meets_specification.meetsSpec);
+    nominals[result.nominal] = count++;
+
+    if (
+      Math.abs(result.nominal - result.measurement) >
+      Math.abs(result.nominal - mostFarMeasurement)
+    ) {
+      mostFarMeasurement = result.measurement;
+    }
 
     return result;
   });
 
-  const baseSpec = results[0];
+  let countOfMostOccuredNominal = Math.max(...Object.values(nominals));
+
+  let mostOccuredNominal = Object.keys(nominals).find(
+    (nominal) => nominals[nominal] === countOfMostOccuredNominal
+  );
+
+  const baseSpec = results.find(
+    (result) => result.nominal === parseInt(mostOccuredNominal)
+  );
   const baseITValue = baseSpec.matched_spec[baseSpec.IT_grade];
 
   const meetsIT = ITDifference <= baseITValue;
-  const itMeetingReason = meetsIT
-    ? `The difference between ${parseToFixedThreeString(
-        largestMeasurement
-      )} and ${parseToFixedThreeString(
-        smallestMeasurement
-      )} is less than or equal to ${baseITValue}.`
-    : `The difference between ${parseToFixedThreeString(
-        largestMeasurement
-      )} and ${parseToFixedThreeString(
-        smallestMeasurement
-      )} is greater than to ${baseITValue}.`;
+  const itMeetingReason = generateReasonForTolerances(
+    meetsIT,
+    largestMeasurement,
+    smallestMeasurement,
+    baseITValue
+  );
 
+  const meetsSpec = withInSpecs.every((v) => v === true);
+  const specMeetingReason = generateReasonForSpecs(
+    meetsSpec,
+    mostFarMeasurement,
+    baseSpec.computed_specification_bounds.lowerBound,
+    baseSpec.computed_specification_bounds.upperBound
+  );
   return {
-    baseSpec,
+    ...baseSpec,
+    measurement: measurements,
+    meets_specification: { meetsSpec, reason: specMeetingReason },
     meets_IT_Tolerance: { meetsIT, reason: itMeetingReason },
+    meets_final_compliance: meetsIT && baseSpec?.meets_specification?.meetsSpec,
   };
 }
 
+function generateReasonForSpecs(spec, measurement, base1, base2) {
+  if (spec === true) {
+    return `${parseToFixedThreeString(
+      measurement
+    )} falls between ${base1} and ${base2}`;
+  }
+  return `${parseToFixedThreeString(
+    measurement
+  )} doesn't fall between ${base1} and ${base2}`;
+}
+
+function generateReasonForTolerances(spec, measurement1, measurement2, base) {
+  if (spec === true) {
+    return `The difference between ${parseToFixedThreeString(
+      measurement1
+    )} and ${parseToFixedThreeString(
+      measurement2
+    )} is less than or equal to ${base}.`;
+  }
+  return `The difference between ${parseToFixedThreeString(
+    measurement1
+  )} and ${parseToFixedThreeString(measurement2)} is greater than ${base}.`;
+}
+
 function validateMeasurements(measurements) {
   if (!Array.isArray(measurements)) {
     return {
@@ -387,14 +436,12 @@ function validateMeasurements(measurements) {
       error: "Measurements array cannot be empty",
     };
   }
-
   return null;
 }
 
 module.exports = {
   getAllTolerancesFor,
   getCamcoStandardTolerancesFor,
-  parseNominalFromMeasurement,
   checkOneMeasurementFor,
   checkMultipleMeasurementsFor,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mechanical-tolerance-calculator",
-  "version": "1.0.5",
+  "version": "1.0.7",
   "description": "Calculates international standard specification and  tolerances for bores, round bars and metals of mechanical units. For examples; H7, H8, H9, h8, h9 specifications and IT5/IT6 tolerances.",
   "main": "index.js",
   "scripts": {