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

Navigation fixes #318

Merged
merged 3 commits into from
Oct 17, 2023
Merged

Navigation fixes #318

merged 3 commits into from
Oct 17, 2023

Conversation

tagener-noisu
Copy link
Collaborator

@tagener-noisu tagener-noisu commented Oct 17, 2023

Some things I fixed when I was making traces.

  1. When bots were trying to get from the base to an objective that was far away they were killed by timeout. It's a problem for large maps like Death Valley, because first objectives are really far away and not all bots were taking vehicles. So I changed to killing them only if they are stuck on the base path only, and not on paths that connect the base with the objectives.
  2. I created single objective paths for vehicles and bots started to spawn on them. Fixed it.
  3. Sometimes bots get stuck between two objectives if their assigned objective is different from those two. Let bots consider to switch to either one of those objectives and since every objective should be connected to every other objective, they will find the path to their assigned objective.

Sometimes bots get stuck between two objectives if their objective is
different from those two. Let bots consider to switch to either one of
those objectives and since every objective should be connected to every
other objective, they will find the path to their desired objective.
@Joe91
Copy link
Owner

Joe91 commented Oct 17, 2023

Wow! Great work! Thanks a lot!

@Joe91 Joe91 merged commit 25632c9 into Joe91:dev Oct 17, 2023
2 checks passed
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

Successfully merging this pull request may close these issues.

2 participants