Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

fix: bale finder retry logic#3136

Merged
Tensuko merged 8 commits intomainfrom
3012-bug_sp-bale-collecting-and-wrapping
Mar 22, 2024
Merged

fix: bale finder retry logic#3136
Tensuko merged 8 commits intomainfrom
3012-bug_sp-bale-collecting-and-wrapping

Conversation

@pvaiko
Copy link
Contributor

@pvaiko pvaiko commented Mar 12, 2024

Retry with another bale if the pathfinding fails with invalid goal, meaning there is likely another bale near the target.

@pvaiko pvaiko self-assigned this Mar 12, 2024
@pvaiko pvaiko force-pushed the 3012-bug_sp-bale-collecting-and-wrapping branch from 0fe765e to c68f1e0 Compare March 12, 2024 20:18
pvaiko added 3 commits March 15, 2024 19:43
Retry with another bale if the pathfinding fails with
invalid goal, meaning there is likely another bale near
the target.
- did not store the last checked bale properly
- obstacle check was not working because missing return

#3012
@pvaiko pvaiko force-pushed the 3012-bug_sp-bale-collecting-and-wrapping branch from c68f1e0 to fd841e2 Compare March 15, 2024 23:45
pvaiko added 5 commits March 16, 2024 08:44
Don't use the pathfinder controller retry logic as we don't
want to retry with the same goal.

Give up after 3 retries. Try another bale if the goal node
is invalid, or failed, but not close to the field edge.

If close to the field edge, back up a bit and start again.

Keep a list of bales tried to avoid retrying the same two bales
and causing a stack overflow.
when everything fails, reverse and try again
Stop retrying also when there are no retryable bales left.
Separate states for reversing due to obstacle or
pathfinder failure.
Rescan field if bumped into another bale while driving
to the target, to make sure it isn't on the list anymore
@Tensuko Tensuko merged commit 6e50aeb into main Mar 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG_SP] Bale collecting and Wrapping, HUD Target Icon opens wrong AI Job

2 participants