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 default "find path" behavior and associated flag. #83

Closed
barrycaceres opened this issue May 20, 2024 · 1 comment
Closed

Change default "find path" behavior and associated flag. #83

barrycaceres opened this issue May 20, 2024 · 1 comment
Assignees
Labels
V4 To be delivered in V4

Comments

@barrycaceres
Copy link
Contributor

barrycaceres commented May 20, 2024

Upon completion of GDEV-3833 task to update the "find path" behavior, then following must occur

  • The parameter name to "find path" function will change from exclusions to avoidances, or avoids (SDK team needs to decide)

  • Called native C functions must use the new C function naming where the word Exclude is replaced with Avoid

  • Documentation of default behavior of the findPath____ functions will needs to reflect that it now treats exclusions/avoidances similar to GPS routing with avoidances. It will only use the avoided entities as a last resort to find a path (route)

  • The SZ_FIND_PATH_PREFER_EXCLUDE flag name will change to SZ_FIND_PATH_STRICT_AVOID and the meaning of the flag will be flipped so that it now forbids the use of avoided entities to find the path

  • Update unit tests to verify the avoidances are handled properly with the new behavior.

@barrycaceres
Copy link
Contributor Author

Resolved

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

No branches or pull requests

3 participants