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.

Sign up for GitHub

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

Jump to bottom

fix: a offmeshlink should alloc 3 dtLinks at start tile and 1 dtLink at end … #622

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: a offmeshlink should alloc 3 dtLinks at start tile and 1 dtLink at end … #622

wants to merge 1 commit into from

Conversation

zkz196
Copy link

@zkz196 zkz196 commented Apr 10, 2023

The number of dtLinks assigned to offmeshlink is incorrect, resulting in loss of connections between polygons

@zkz196 zkz196 changed the title a offmeshlink will alloc 3 dtLinks at start tile and 1 dtLink at end … fix: a offmeshlink should alloc 3 dtLinks at start tile and 1 dtLink at end … Apr 10, 2023
@grahamboree
Copy link
Member

Hi! Thanks for the PR. Can you provide some more information about the specific issue this is solving?

@zkz196
Copy link
Author

zkz196 commented Apr 11, 2023

image
as you see, the old code calculated maxLinkCount as
edgeCount(4) + portalCount(2)*2 + offMeshConLinkCount(8)*2 = 24.
In fact, offMeshLink will be used
baseOffMeshLinks(8) * 2 + connectExtOffMeshLinks(8) = 24
so the start polygon do not link the end polygon directly

@grahamboree grahamboree self-assigned this Apr 20, 2023
@grahamboree grahamboree removed their assignment Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
area: Detour type: bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@zkz196 @grahamboree