barefoot icon indicating copy to clipboard operation
barefoot copied to clipboard

Changeable radius while running instance

Open dvent7 opened this issue 6 years ago • 1 comments

Hello,

I wanted to change the search radius for roads in special cases. e.g. tunnels/bridges or beacons. But when I have multiple threads which are using the https://github.com/bmwcarit/barefoot/blob/master/src/main/java/com/bmwcarit/barefoot/matcher/Matcher.java class, then setting the radius will have also effects on other threads which are using the same map/matcher.

Therefore I suggest to add an optional additonal parameter for the Method execute https://github.com/bmwcarit/barefoot/blob/c7b55daab5fe14c3e0b02f9e76e22ca509633ac4/src/main/java/com/bmwcarit/barefoot/markov/Filter.java#L114 which uses this parameter for finding candidates (https://github.com/bmwcarit/barefoot/blob/c7b55daab5fe14c3e0b02f9e76e22ca509633ac4/src/main/java/com/bmwcarit/barefoot/markov/Filter.java#L127)

dvent7 avatar Nov 08 '17 13:11 dvent7

Sorry for the delay. Is this still an open issue? The radius could be adapted based on the samples. I see that the abstract Filter class may be a design flaw for this use case but you could still derive the Matcher class and adapt the radius based on properties of the sample. If you need more details on how to accomplish this please let me know.

smattheis avatar Dec 06 '17 17:12 smattheis