Merges test datasets: keeps Berlin, removes Monaco

Author: daniel-j-h

.gitignore

@@ -46,7 +46,7 @@ Thumbs.db
 #######################
 /build/
 /example/build/
-/test/data/monaco*
+/test/data/berlin*
 /test/bindings/node/data/berlin*
 /cmake/postinst
 
@@ -87,7 +87,7 @@ stxxl.errlog
 /test/cache
 /test/speeds.csv
 /test/penalties.csv
-/test/data/monaco.*
+/test/data/berlin.*
 node_modules
 
 # Deprecated config file #

.travis.yml

@@ -198,10 +198,10 @@ script:
   - echo "travis_fold:start:BENCHMARK"
   - make -C test/data benchmark
   - echo "travis_fold:end:BENCHMARK"
-  - ./example/build/osrm-example test/data/monaco.osrm
+  - ./example/build/osrm-example test/data/berlin.osrm
   # All tests assume to be run from the build directory
   - pushd ${OSRM_BUILD_DIR}
-  - ./unit_tests/library-tests ../test/data/monaco.osrm
+  - ./unit_tests/library-tests ../test/data/berlin.osrm
   - ./unit_tests/extractor-tests
   - ./unit_tests/engine-tests
   - ./unit_tests/util-tests

docs/testing.md

@@ -49,7 +49,7 @@ You should see the compiled binaries in `build/unit_tests`, you can then run eac
 For `library-tests` you will need to provide a path to the test data:
 
 ```
-./library-tests ../../test/data/monaco.osrm
+./library-tests ../../test/data/berlin.osrm
 ```
 
 ## Cucumber

example/example.cpp

@@ -39,9 +39,9 @@ int main(int argc, const char *argv[])
     // The following shows how to use the Route service; configure this service
     RouteParameters params;
 
-    // Route in monaco
-    params.coordinates.push_back({util::FloatLongitude{7.419758}, util::FloatLatitude{43.731142}});
-    params.coordinates.push_back({util::FloatLongitude{7.419505}, util::FloatLatitude{43.736825}});
+    // Route in Berlin: Alexanderplatz to Hackescher Markt
+    params.coordinates.push_back({util::FloatLongitude{13.414307}, util::FloatLatitude{52.521835}});
+    params.coordinates.push_back({util::FloatLongitude{13.402290}, util::FloatLatitude{52.523728}});
 
     // Response is in JSON format
     json::Object result;
@@ -59,7 +59,7 @@ int main(int argc, const char *argv[])
         const auto duration = route.values["duration"].get<json::Number>().value;
 
         // Warn users if extract does not contain the default Berlin coordinates from above
-        if (distance == 0 or duration == 0)
+        if (distance == 0 || duration == 0)
         {
             std::cout << "Note: distance or duration is zero. ";
             std::cout << "You are probably doing a query outside of the OSM extract.\n\n";

example/example.js

@@ -5,10 +5,10 @@ var OSRM = require('..');
 var path = require('path');
 
 var app = express();
-var osrm = new OSRM(path.join(__dirname,"../test/data/monaco.osrm"));
+var osrm = new OSRM(path.join(__dirname,"../test/data/berlin.osrm"));
 
 // Accepts a query like:
-// http://localhost:8888?start=13.438640,52.519930&end=13.415852,52.513191
+// http://localhost:8888?start=13.414307,52.521835&end=13.402290,52.523728
 app.get('/', function(req, res) {
     if (!req.query.start || !req.query.end) {
         return res.json({"error":"invalid start and end query"});

scripts/poly2req.js

@@ -51,8 +51,8 @@ else if (process.argv.length > 2 && process.argv[2] == "dc")
 }
 else if (process.argv.length > 2)
 {
-  let monaco_poly_path = process.argv[2];
-  let poly_data = fs.readFileSync(monaco_poly_path, 'utf-8');
+  let poly_path = process.argv[2];
+  let poly_data = fs.readFileSync(poly_path, 'utf-8');
 
   // lets assume there is only one ring
   // cut of name and ring number and the two END statements

src/benchmarks/match.cpp

@@ -1,4 +1,4 @@
-DATA_NAME:=monaco
+DATA_NAME:=berlin
 DATA_URL:=https://s3.amazonaws.com/mapbox/osrm/testing/$(DATA_NAME).osm.pbf
 DATA_POLY_URL:=https://s3.amazonaws.com/mapbox/osrm/testing/$(DATA_NAME).poly
 OSRM_BUILD_DIR?=../../build

test/data/Makefile

@@ -1,2 +1,2 @@
-2b8dd9343d5e615afc9c67bcc7028a63  monaco.osm.pbf
-b0788991ab3791d53c1c20b6281f81ad  monaco.poly
+54707104e9f411d1a3032c00fb1276c6  berlin.osm.pbf
+031d988f23fc2a30371c9ef8d2a49bae  berlin.poly

test/data/data.md5sum

@@ -4,5 +4,5 @@ if (process.env.OSRM_DATA_PATH !== undefined) {
     exports.data_path = path.join(path.resolve(process.env.OSRM_DATA_PATH),"berlin-latest.osrm");
     console.log('Setting custom data path to ' + exports.data_path);
 } else {
-    exports.data_path = path.resolve(path.join(__dirname,"data/berlin-latest.osrm"));
+    exports.data_path = path.resolve(path.join(__dirname, "../data/berlin-latest.osrm"));
 }

test/nodejs/data/Makefile

@@ -5,7 +5,7 @@
 
 #include <vector>
 
-// Somewhere in 2b8dd9343d5e615afc9c67bcc7028a63 Monaco
+// Somewhere in d41d8cd98f00b204e9800998ecf8427e Berlin
 
 // Convenience aliases
 using Longitude = osrm::util::FloatLongitude;
@@ -15,21 +15,21 @@ using Locations = std::vector<Location>;
 
 inline Location get_dummy_location()
 {
-    return {osrm::util::FloatLongitude{7.437069}, osrm::util::FloatLatitude{43.749249}};
+    return {osrm::util::FloatLongitude{13.388860}, osrm::util::FloatLatitude{52.517037}};
 }
 
 inline Locations get_locations_in_small_component()
 {
-    return {{Longitude{7.438023}, Latitude{43.746465}},
-            {Longitude{7.439263}, Latitude{43.746543}},
-            {Longitude{7.438190}, Latitude{43.747560}}};
+    return {{Longitude{13.459765}, Latitude{52.543193}},
+            {Longitude{13.461455}, Latitude{52.542381}},
+            {Longitude{13.462940}, Latitude{52.541774}}};
 }
 
 inline Locations get_locations_in_big_component()
 {
-    return {{Longitude{7.415800}, Latitude{43.734132}},
-            {Longitude{7.417710}, Latitude{43.736721}},
-            {Longitude{7.421315}, Latitude{43.738814}}};
+    return {{Longitude{13.442631}, Latitude{52.551110}},
+            {Longitude{13.441193}, Latitude{52.549506}},
+            {Longitude{13.439648}, Latitude{52.547705}}};
 }
 
 #endif