Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Include map matching module from external repository #1334

Closed
wants to merge 225 commits into from
Closed

Include map matching module from external repository #1334

wants to merge 225 commits into from

Conversation

karussell
Copy link
Member

@karussell karussell commented Apr 11, 2018 •
edited

Used the stackoverflow answer recommended from @stefanholder to get it nicely merged in a subdirectory with all the 216 commits: https://stackoverflow.com/a/1426163/194609

  • integrate Java library as map-matching module (probably just copying folder matching-core) and add matching-web code to web. And hmm-lib should be probably stay named as it.
  • dropwizardify
  • merge documentation and contributor list
  • integrate map-matching.sh script into graphhopper.sh and use MapMatchingMain as base for new dropwizard app class
  • what to do with the UI? Either include it in webdebug and serve files from disc or bundle assets somehow but under /map-matching-demo or similar

Peter and others added 30 commits December 9, 2014 23:08
initial version with bugs, tests not passing
5fd0f2f
created map-matching.sh script
5d26303
minor readme change
5c3183c
minor readme change
66b6fde
added blog post link
6543e43
added alpha warning
a990798
support milisecond gpx timestamps
c3f8832
minor refactoring
0ce0a21
gitignore
eaec85f
@karussell
Merge pull request #4 from rory/support-subsecond-gpx
3fc16df 
Support milisecond gpx timestamps
added test files and tests for GPXFile import
5f35626
fixed all tests and improved location lookup, although it could be sl…
07e72d9 
…ower
removed failed test from readme
2506832
add travis ci
f40c2e0
@karussell
Update README.md
088f8b1
fixed failing test
d8facbb
included travis pic
6b4d5c5
added use case
4a6b0b5
referenced picture for illustration
bb684bf
made it possible to reuse existing index from any initialized GraphHo…
bf6059f 
…pper
fixing #5, avoid virtual edges in public response of MapMatching.doWork
8286983
added test to ensure no virtual edge is returned, #5
0a2002f
first working version without much trouble
1a1cdf4
fix for stopwatch
38f3bae
no sys out in tests
bd358d0
minor tweak to improve matching at merging points
9cfdc3a
@karussell
Merge pull request #7 from graphhopper/issue_1
bc7bdc6 
This PR fixes the problem when loops occur.
make gpx accuracy filter configurable
99f19a8
implemented a new forceRepair=true
73b2ed3
@ustroetz
add possible flags to readme
5261bfc
@karussell karussell added this to the 0.11 milestone Apr 11, 2018
@karussell karussell mentioned this pull request Apr 12, 2018
Closed
@karussell karussell removed this from the 0.11 milestone May 16, 2018
@karussell karussell closed this Jun 8, 2018
@karussell karussell deleted the map_matching_include branch June 8, 2018 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
architecture new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
10 participants
@karussell @ustroetz @tyrasd @chucre @devemux86 @tochev @stefanholder @kodonnell @michaz @hbruch