-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
Per-layer cost coefficients #78
Conversation
Actually, it doesn't work. I've identified a few problems:
I'm going to close this PR. I will open a new one if I'm able to find a solution. |
the at the moment the "best" place where it's documented is this test: https://github.com/vleue/polyanya/blob/d1102969b3bb98af303f94c8eec64baa7ab926f6/src/stitching.rs#L899C8-L899C34 Could you give more info about the issues you had? |
Yeah I saw the comment but when I tried |
Adds support for per-layer cost coefficients.
For the heuristic, I used
min-layer-cost
as this should be both admissible and backwards-compatible, though I'm sure better heuristics are possible.I need this functionality for my own use, though I see it's a commonly-requested feature, e.g. #35, vleue/vleue_navigator#66, and vleue/vleue_navigator#46
I saw in the docs that
layer.scale
can be used as a cost-coefficient, but I found it hard to work with, due to some unwanted interaction with mesh scaling and obstacle scaling. Thiscost
coefficient should be more straightforward to use.