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.

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

Change of the meaning for 'dynamic range' #762

Open
afischerdev opened this issue Feb 16, 2025 · 2 comments
Open

Change of the meaning for 'dynamic range' #762

afischerdev opened this issue Feb 16, 2025 · 2 comments

Comments

@afischerdev
Copy link
Collaborator

The dynamic range was introduced with PR #729 and #747 and enabled the issue #387 - this also carries the description.

The work on the round trip (#759) shows that the dynamic range could be useful in other contexts as well.

But this reduces the meaning to the original meaning of the words:
find the next point that is connected to a way.

The current meaning has the addition
and add a beeline

To play with the problem I added a test library to the test server.

New is the parameter for the profile in global section:

assign add_beeline          = true    # default is false

Please see this sample 2 points or sample 3 points

Now if you add the new beeline parameter above, you will see it in the result.

To get all examples used in #387 running there is one more parameter:

# car profile
assign use_offroad          = true       # default is false

This is because we used use_dynamic_range also to enable some other roads

assign caraccess ...
    switch and highway=track use_offroad     1
    switch and highway=road use_offroad      1

Should be clearer now.

@devemux86
Copy link
Contributor

@afischerdev Thanks, very nice improvement. 👍

Will you also include it in a pull request?

@afischerdev
Copy link
Collaborator Author

afischerdev commented Feb 17, 2025

@devemux86
Yes, that is the plan. Because it is also needed for roundtrip #759 to avoid ambiguity.

But there should be a little time for testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants