Repository navigation
Snake: look at every pass nearer than a far order pass - #228
Merged
Merged
Conversation
With skip mode "Ignore worked tracks", when the snake order's next pass was more than twice the skip's step away, the nearest pass still to do was looked for only within twice the skip's step, and then the order's own pass, further over, was tried. On a tester's field (simulator) the tractor on 21R had the order's 8R 13 passes over and 15R six over: 15R was never looked at, the turn onto 8R did not build, and there was no U-turn (AutoSteer went off at the end of the pass). Asked before a far order pass, the search now reaches every pass nearer than the order's, either side. The fallback after the order's pass has no room or no turn keeps its limit of twice the skip's step. On the tester's field the turn from 21R onto 15R builds, inside the fence. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VKEBDFmQpFVsahedjtN2AZ
3 tasks done
renatorgr
marked this pull request as ready for review
November 14, 2026 11:05
This was referenced Nov 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by Renato · project thread
What changed
With skip mode Ignore worked tracks, when the snake order's next pass is more than twice the skip's step away (
MaxFallbackStep), the nearest pass still to do is tried first (#223). That search only looked within twice the skip's step, and then the order's own pass, further over, was tried.On Tapada da Casa (simulator, beta 2be8132, 14 Nov 10:01) the tractor was on 21R. The order's next pass was 8R, 13 passes over; 15R, 6 over, was still to do. 15R was never looked at, the turn onto 8R did not build ("Turn near hard boundary was malformed"), so there was no U-turn and AutoSteer went off at the end of the pass (#197, correct on its own).
Now, when asked before a far order pass,
TurnOntoNearestOpenPassreaches every pass nearer than the order's, on either side. The fallback after the order's pass has no room or no turn keeps its limit of twice the skip's step.Replaying the dump's field and state (passes worked as in the dump, 8 m front boom, radius 8 m, U-turn distance 2 m, hard boundary): before, the result was 21R -> 8R, malformed, no turn. After, the result is 21R -> 15R: a 163-point turn along the headland, every point inside the fence, with the implement 2.28 m from the fence.
Tests (
SnakeOpenPassTests):CLAUDE.md's snake bullet says so. Fork only.
Related issue
Follows #223 (nearest open pass before a far order pass).
Pre-submit checklist
dotnet buildsucceeds with no errorsdotnet testpasses: the U-turn tests (FullyQualifiedName~YouTurn, 1722 passed); the full suite runs in CIScreenshots
Not a UI change.
Generated by Claude Code