landxml 0.4.1 → 0.4.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # landxml
 
+## 0.4.2
+
+### Patch Changes
+
+- fbd9682: Contour elevations from surface min/max elevations and increment are now calculated correctly (bugfix).
+
 ## 0.4.1
 
 ### Patch Changes

package/dist/index.js

@@ -281,11 +281,14 @@ var linesToPolyLines = (lineSegments) => {
   return polylines;
 };
 var contourElevations = (minElevation, maxElevation, interval) => {
-  if (maxElevation >= Infinity) {
+  if (!Number.isFinite(minElevation) || !Number.isFinite(maxElevation) || !Number.isFinite(interval)) {
     throw new Error("Contour elevations have to be finite numbers");
   }
+  if (minElevation + interval > maxElevation) {
+    throw new Error(`No contour lines at interval: ${interval} between elevation ${minElevation} and ${maxElevation}`);
+  }
   const elevations = [];
-  let elevation = Math.ceil(minElevation * interval) / interval;
+  let elevation = Math.ceil(minElevation / interval) * interval;
   while (elevation < maxElevation) {
     elevations.push(elevation);
     elevation += interval;

package/dist/index.mjs

@@ -246,11 +246,14 @@ var linesToPolyLines = (lineSegments) => {
   return polylines;
 };
 var contourElevations = (minElevation, maxElevation, interval) => {
-  if (maxElevation >= Infinity) {
+  if (!Number.isFinite(minElevation) || !Number.isFinite(maxElevation) || !Number.isFinite(interval)) {
     throw new Error("Contour elevations have to be finite numbers");
   }
+  if (minElevation + interval > maxElevation) {
+    throw new Error(`No contour lines at interval: ${interval} between elevation ${minElevation} and ${maxElevation}`);
+  }
   const elevations = [];
-  let elevation = Math.ceil(minElevation * interval) / interval;
+  let elevation = Math.ceil(minElevation / interval) * interval;
   while (elevation < maxElevation) {
     elevations.push(elevation);
     elevation += interval;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "landxml",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "description": "Parse LandXML surfaces on the modern web.",
   "main": "dist/index.js",
   "module": "dist/index.mjs",