{"action":"created","issue":{"url":"https://api.github.com/repos/graphhopper/map-matching/issues/73","repository_url":"https://api.github.com/repos/graphhopper/map-matching","labels_url":"https://api.github.com/repos/graphhopper/map-matching/issues/73/labels{/name}","comments_url":"https://api.github.com/repos/graphhopper/map-matching/issues/73/comments","events_url":"https://api.github.com/repos/graphhopper/map-matching/issues/73/events","html_url":"https://github.com/graphhopper/map-matching/pull/73","id":181049051,"number":73,"title":"Performance measurement suite","user":{"login":"kodonnell","id":8204904,"avatar_url":"https://avatars.githubusercontent.com/u/8204904?v=3","gravatar_id":"","url":"https://api.github.com/users/kodonnell","html_url":"https://github.com/kodonnell","followers_url":"https://api.github.com/users/kodonnell/followers","following_url":"https://api.github.com/users/kodonnell/following{/other_user}","gists_url":"https://api.github.com/users/kodonnell/gists{/gist_id}","starred_url":"https://api.github.com/users/kodonnell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kodonnell/subscriptions","organizations_url":"https://api.github.com/users/kodonnell/orgs","repos_url":"https://api.github.com/users/kodonnell/repos","events_url":"https://api.github.com/users/kodonnell/events{/privacy}","received_events_url":"https://api.github.com/users/kodonnell/received_events","type":"User","site_admin":false},"labels":[{"id":455461252,"url":"https://api.github.com/repos/graphhopper/map-matching/labels/architecture","name":"architecture","color":"006b75","default":false},{"id":455461611,"url":"https://api.github.com/repos/graphhopper/map-matching/labels/improvement","name":"improvement","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":23,"created_at":"2016-10-05T02:07:25Z","updated_at":"2016-12-20T21:36:38Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/graphhopper/map-matching/pulls/73","html_url":"https://github.com/graphhopper/map-matching/pull/73","diff_url":"https://github.com/graphhopper/map-matching/pull/73.diff","patch_url":"https://github.com/graphhopper/map-matching/pull/73.patch"},"body":"As discussed in #67 and #68. Code isn't ready for a full merge - I'm looking for some initial feedback on the general approach. Key points:\n- created `matching-tools` module (like `graphhopper-tools`)\n- copied `Measurement.java` from GH, and added two tests: one for location lookups (i.e. `findNClosest`) and one for testing a bunch of randomly created GPX entries (randomly pick start/end, then find route, then down-sample points).\n- learned (some of) Maven, the hard way\n\nTodo:\n- general tidy up\n- tweak random creation of GPX entries (above) to be near the path, not on it.\n- do I need to warm up the JVM?\n- ...?\n\nExample usage:\n\n```\n$ ./map-matching.sh action=measurement map-data/leipzig_germany.osm.pbf 3\n...\n$ cat measurement.20161005_145346 \ncommits (in order tested):\n3b305dc8a76123a12375da91a69d8ecf47466ed0 merged most recent\ncacb7a3e0d2033042ca11d7280af43501a2a06dc ok, seems to be working ...\n59a01fcc3307a05e9b8e74119ec1948f8a9de7c9 getting multiple commits working ...\n\nmeasurements:\n-------------\n\nlocation_index_match.max      5.978906            6.654648            6.393774            \nlocation_index_match.mean     0.1895065804        0.194572196         0.1902064926        \nlocation_index_match.min      0.001334            0.00126             0.001377            \nlocation_index_match.sum      947.532902          972.86098           951.032463          \nmap_match.max                 8.049864            7.153459            7.29994             \nmap_match.mean                0.7924976776        0.7868942498        0.7843470472        \nmap_match.min                 0.094859            0.128563            0.091647            \nmap_match.sum                 3962.488388         3934.471249         3921.735236         \nmeasurement.count             5000                5000                5000                \nmeasurement.seed              123                 123                 123                 \nmeasurement.time              7010                7007                7027                \nmeasurement.totalMB           655                 655                 655                 \nmeasurement.usedMB            19                  19                  21                  \n```\n"},"comment":{"url":"https://api.github.com/repos/graphhopper/map-matching/issues/comments/268366260","html_url":"https://github.com/graphhopper/map-matching/pull/73#issuecomment-268366260","issue_url":"https://api.github.com/repos/graphhopper/map-matching/issues/73","id":268366260,"user":{"login":"kodonnell","id":8204904,"avatar_url":"https://avatars.githubusercontent.com/u/8204904?v=3","gravatar_id":"","url":"https://api.github.com/users/kodonnell","html_url":"https://github.com/kodonnell","followers_url":"https://api.github.com/users/kodonnell/followers","following_url":"https://api.github.com/users/kodonnell/following{/other_user}","gists_url":"https://api.github.com/users/kodonnell/gists{/gist_id}","starred_url":"https://api.github.com/users/kodonnell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kodonnell/subscriptions","organizations_url":"https://api.github.com/users/kodonnell/orgs","repos_url":"https://api.github.com/users/kodonnell/repos","events_url":"https://api.github.com/users/kodonnell/events{/privacy}","received_events_url":"https://api.github.com/users/kodonnell/received_events","type":"User","site_admin":false},"created_at":"2016-12-20T21:36:38Z","updated_at":"2016-12-20T21:36:38Z","body":"> If that is too much for the beginning I can do it :)\r\n\r\nYes please! It looked simple enough, except I've also got merges in there, and for now I'd appreciate if you [dealt with those](http://stackoverflow.com/questions/37981053/how-to-use-git-rebase-to-squash-commits-around-a-merge), if it makes sense. (If it were a private repo, I might play, but I don't want to muck this up for others.) I'm happy for you to squash it all into a single commit (if possible). Thanks!"}}
{"actor":{"display_login":"kodonnell"}}