Section Control fixes #19
Merged
Conversation
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
When the activity is suspended (e.g. AUX-N master switch) the painting is also suspended in AgOpenGPS.
… into gunicsba/section-fixes
…üller Jobrechner ECU.
…ement. Fixed the parameter names in the help function of the console.
…onfusing that it works with 2 boolean values at the same time.
…onfusing that it works with 2 boolean values at the same time. + Fix formatting
…AOG-TaskController into gunicsba/section-fixes
…onfusing that it works with 2 boolean values at the same time.
…AOG-TaskController into gunicsba/section-fixes
…nt 0 (cherry picked from commit d506175)
The planter seems to recognize it and every time we tried to set the value for it all of its sections turned on.
gunicsba
commented
Dec 30, 2025
GwnDaan
requested changes
Dec 30, 2025
Collaborator
GwnDaan
left a comment
GwnDaan
left a comment
There was a problem hiding this comment.
You're going in the right direction for sure, just needs to improve a bit to make it robust and ready for the future
…I and returns whether it has the Settable property. Added CondensedSectionOverrideState (DDI 291-296) as it was useful for debugging Simplify SetpointWorkState error handling - Consolidated error logging to only report when DDI 289 is not available, as there are some implement that do not use it.
GwnDaan
approved these changes
Jan 3, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Eliminated the 16 section problem raised by Patrick and reported by Ryan. (And this should also work with 32,48 etc sections)
#15
Implemented the legacy section control changes required by Damman. Unfortunately I wasn't able to test myself right now as I don't posess a machine.
Increased the CTS packet count so the startup is a bit nicer.
Fixed the typos with the console parameter names.