Repository navigation
Snake: skip passes with no crop at this end when the headland is on - #229
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
Since the U-turn reads the turn line (the fence moved in by the U-turn distance), the snake's landing check found room on a pass lying in the headland band, where the sections are off: on a tester's field (simulator) it drove 13L, with 12 m of crop at its far end, and 15L, with none, in a corner of the headland, and only then went back to 12L. With the operator's headland on and the sections off in it, a pass the snake turns onto (the order's or a nearest open one) must also have the landing room inside the headland line (SnakeLandable). Without a headland, or with the sections on in it, nothing changes. Fork only. 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:24
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
Stacked on #228: merge #228 first. This branch has #228's commit under its own, so until #228 is merged the diff shows both.
What changed
Since official AgOpenGPS-Official#816 the U-turn reads the turn line (the fence moved in by the U-turn distance), and so does the snake's landing check (
IsPathLandable: 20 m of the next pass inside the area at this end). With the operator's headland on and the sections off in it, a pass lying in the headland band has that room on the turn line and nothing to work: unpainted, it is still to do, so the snake turned onto it.On a tester's field (simulator, beta 2be8132, 14 Nov 10:12) the snake drove 13L, with 12 m of crop at its far end, and 15L, with none, in a corner of the headland, and only then went back to 12L.
Now a pass the snake turns onto (the order's next pass, or a nearest open one from
TurnOntoNearestOpenPass) must also have the landing room inside the headland line when the headland is on andTool.IsHeadlandSectionControlis set (SnakeLandable). Without a headland, or with the sections on in it, nothing changes: the turn line alone decides, as before.Tests (
SnakeOpenPassTests):CLAUDE.md's snake bullet says so. Fork only.
Related issue
Follows #228 (nearest open pass before a far order pass), from the same day's simulator reports.
Pre-submit checklist
dotnet buildsucceeds with no errorsdotnet testpasses: the U-turn, snake and headland tests (1758 passed); the full suite runs in CIScreenshots
Not a UI change.
Generated by Claude Code