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

fix:rcBuildLayerRegions missing areaType #470

Conversation

shineTeam7
Copy link
Contributor

fix:rcBuildLayerRegions missing areaType

@grahamboree grahamboree removed their assignment Jul 4, 2023
@jankrassnigg
Copy link

This fix is absolutely vital to get area IDs working correctly. Here is what happens without it:

grafik

The black blocks use a different area ID and should show up in red rather than blue. As you can see, the blue area ID takes over in many places. The dark blue grid lines are tile borders, so that's why the area ID is correct for half the black block, but not the other half, because a tile border separates them.

With the fix, the area IDs work as expected:

grafik

It would be great, if this PR could be merged to the main branch.

@grahamboree grahamboree self-assigned this Sep 7, 2023
@grahamboree grahamboree merged commit ddaa361 into recastnavigation:main Nov 12, 2023
zlonglin pushed a commit to zlonglin/recastnavigation that referenced this pull request Feb 1, 2024
Co-authored-by: sunming <359944951@qq.com>
Co-authored-by: Graham Pentheny <grahamboree@users.noreply.github.com>
(cherry picked from commit ddaa361)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants