@wemap/osm 2.7.1 → 2.7.4

package/assets/bureaux-wemap-montpellier-network.osm

@@ -108,9 +108,8 @@
     <nd ref='-440488' />
     <nd ref='-440490' />
     <nd ref='-440492' />
-    <tag k='highway' v='footway' />
+    <tag k='highway' v='steps' />
     <tag k='level' v='1;2' />
-    <tag k='stairs' v='yes' />
   </way>
   <way id='-440503' action='modify' visible='true'>
     <nd ref='-440484' />

package/package.json

@@ -11,7 +11,7 @@
         "directory": "packages/osm"
     },
     "name": "@wemap/osm",
-    "version": "2.7.1",
+    "version": "2.7.4",
     "bugs": {
         "url": "https://github.com/wemap/wemap-modules-js/issues"
     },
@@ -26,11 +26,11 @@
     ],
     "license": "ISC",
     "dependencies": {
-        "@wemap/geo": "^2.7.1",
-        "@wemap/graph": "^2.7.1",
+        "@wemap/geo": "^2.7.4",
+        "@wemap/graph": "^2.7.4",
         "@wemap/logger": "^2.7.0",
         "lodash.isnumber": "^3.0.3",
         "sax": "^1.2.4"
     },
-    "gitHead": "3d332f1dce308ba28f2a551c2e709e840c54bd83"
+    "gitHead": "346dd50c4934b77b831a081f3543405865f1a65e"
 }

package/src/network/OsmNetwork.js

@@ -51,11 +51,11 @@ class OsmNetwork extends Network {
             if (node.data.tags.highway === 'elevator') {
                 // We have to clone this node for each connected edge
                 OsmNetwork.createNodesAndEdgesFromElevator(networkModel, node);
-            } else {
-                node.generateLevelFromEdges();
             }
         });
 
+        Node.generateNodesLevels(networkModel.nodes);
+
         return networkModel;
     }
 

package/src/network/OsmRouter.spec.js

@@ -58,13 +58,16 @@ describe('OsmRouter - Multi-level itinerary', () => {
     it('Verify steps', () => {
 
         const steps = itinerary.steps;
-        expect(steps.length).equal(10);
+        expect(steps.length).equal(8);
 
         verifyNodesOrder(steps[0].nodes, [null, 'p7', 'p8']);
-        for (let i = 2; i <= 9; i++) {
-            verifyNodesOrder(steps[i - 1].nodes, ['p' + [i + 6], 'p' + [i + 7]]);
-        }
-        verifyNodesOrder(steps[9].nodes, ['p16']);
+        verifyNodesOrder(steps[1].nodes, ['p8', 'p9']);
+        verifyNodesOrder(steps[2].nodes, ['p9', 'p10']);
+        verifyNodesOrder(steps[3].nodes, ['p10', 'p11']);
+        verifyNodesOrder(steps[4].nodes, ['p11', 'p12', 'p13', 'p14']);
+        verifyNodesOrder(steps[5].nodes, ['p14', 'p15']);
+        verifyNodesOrder(steps[6].nodes, ['p15', 'p16']);
+        verifyNodesOrder(steps[7].nodes, ['p16']);
     });
 
 

package/src/osrm/OsrmUtils.js

@@ -42,7 +42,8 @@ class OsrmUtils {
         for (let i = 0; i <= lastStepId; i++) {
 
             const {
-                nodes, length, nextEdgeData, nextBearing, previousBearing, angle, node, duration
+                nodes, length, nextEdgeData, nextBearing, previousBearing, angle, node,
+                duration, levelChange
             } = itinerarySteps[i];
 
             const edgeData = i !== lastStepId ? nextEdgeData : itinerary.edges[lastStepId];
@@ -71,6 +72,10 @@ class OsrmUtils {
                 wemap: {}
             };
 
+            if (levelChange !== null) {
+                osrmStep.levelChange = levelChange;
+            }
+
             if (node.data && node.data.tags) {
                 osrmStep.wemap.nodeTags = node.data.tags;
             }
@@ -250,17 +255,18 @@ class OsrmUtils {
                 : deg2rad(jsonStep.maneuver.bearing_after);
             step.angle = diffAngle(step.previousBearing, step.nextBearing + Math.PI);
             step.name = jsonStep.name;
+            step.levelChange = jsonStep.levelChange ? jsonStep.levelChange : null;
 
             /**
              * Wemap fields
              */
             if (jsonStep.wemap) {
-                const firstNode = itinerary.nodes[0];
+                const firstNode = step.nodes[0];
                 if (jsonStep.wemap.nodeTags && firstNode) {
                     firstNode.data = { tags: jsonStep.wemap.nodeTags };
                 }
 
-                const firstEdge = itinerary.edges[0];
+                const firstEdge = step.edges[0];
                 if (jsonStep.wemap.nextEdgeTags && firstEdge) {
                     firstEdge.data = {
                         tags: jsonStep.wemap.nextEdgeTags,

package/src/osrm/OsrmUtils.spec.js

@@ -172,28 +172,23 @@ describe('OsrmUtils', () => {
         expect(steps[3].maneuver.bearing_after).be.closeTo(281.93, 0.01);
 
         expect(steps[4].maneuver.modifier).equals('right');
-        expect(steps[4].distance).be.closeTo(4.51, 0.01);
+        expect(steps[4].distance).be.closeTo(10.71, 0.01);
         expect(steps[4].maneuver.bearing_after).be.closeTo(12.22, 0.01);
+        expect(steps[4].levelChange).is.not.null;
+        expect(steps[4].levelChange.direction).equals('down');
+        expect(steps[4].levelChange.diff).equals(-1);
 
-        expect(steps[5].maneuver.modifier).equals('right');
-        expect(steps[5].distance).be.closeTo(1.72, 0.01);
-        expect(steps[5].maneuver.bearing_after).be.closeTo(102.22, 0.01);
+        expect(steps[5].maneuver.modifier).equals('left');
+        expect(steps[5].distance).be.closeTo(4.23, 0.01);
+        expect(steps[5].maneuver.bearing_after).be.closeTo(102.00, 0.01);
 
         expect(steps[6].maneuver.modifier).equals('right');
-        expect(steps[6].distance).be.closeTo(4.49, 0.01);
-        expect(steps[6].maneuver.bearing_after).be.closeTo(192.22, 0.01);
+        expect(steps[6].distance).be.closeTo(5.03, 0.01);
+        expect(steps[6].maneuver.bearing_after).be.closeTo(191.75, 0.01);
 
         expect(steps[7].maneuver.modifier).equals('left');
-        expect(steps[7].distance).be.closeTo(4.23, 0.01);
-        expect(steps[7].maneuver.bearing_after).be.closeTo(102.00, 0.01);
-
-        expect(steps[8].maneuver.modifier).equals('right');
-        expect(steps[8].distance).be.closeTo(5.03, 0.01);
-        expect(steps[8].maneuver.bearing_after).be.closeTo(191.75, 0.01);
-
-        expect(steps[9].maneuver.modifier).equals('left');
-        expect(steps[9].distance).be.closeTo(0, 0.01);
-        expect(steps[9].maneuver.bearing_after).be.closeTo(0, 0.01);
+        expect(steps[7].distance).be.closeTo(0, 0.01);
+        expect(steps[7].maneuver.bearing_after).be.closeTo(0, 0.01);
 
     });