Repository navigation
Seeder 0.2.5: "Sections on the seed drill" setting, 19 sections shown before the program talks - #124
Merged
Merged
Conversation
…em before the program talks The seed drill page drew only the sections the program had asked for since the start (secTop), at least 16, so a module that had not heard AgOpenWeb yet (or just restarted) showed 16 sections instead of the drill's 19. A new setting on the Settings tab, "Sections on the seed drill" (0-64, 0 = as many as the program asks for, kept in flash as "secs"), tells the page how many sections to draw from the start; a relay that would follow a section past the last one does nothing, so the relay card now reads "24 relays, 19 of them sections". /api/settings and /api/status carry "secs"; /api/relays takes it with "first". Simulator: the pretend module has the setting at 19; the smoke test checks the Settings value, relay 20 idle and the 19 sections with the cable never plugged in (?wifi=ap). READMEs: changelog rows and the setup step. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VKEBDHD5gzEhTiu3KLnBSx
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 changes
The seed drill page drew only the sections the program had asked for since the module started (
secTop), and never fewer than 16. A module that had not heard AgOpenWeb yet, or had just restarted, showed 16 sections instead of the drill's 19; the 19 appeared only once AgOpenWeb opened section 19. The simulator showed the same with "tractor network not set".Seeder_ESP32 0.2.5: a new setting on the Settings tab, Sections on the seed drill (0-64; 0 = as many as the program asks for, the old behaviour). With it at 19:
secs), sent in/api/settingsand/api/status, saved by/api/relaystogether with Relay 1 follows section.Simulator: the pretend module has the setting at 19; the smoke test checks the value on the Settings tab, relay 20 idle, and the 19 sections with the cable never plugged in (
seeder/?wifi=ap). READMEs: changelog rows and a setup step.How to test
mainafter the merge): openseeder/?wifi=ap. The Sowing tab shows 19 sections, none open. On the Settings tab the new field reads 19; set it to 0 and save (passwordagopengps): the Sowing tab goes back to 16.Checks
BOARD_RELAY2485 %,BOARD_KC868_A3287 % of flash), no warnings. Not run on a board.build.py+smoke-test.js: "Simulator pages OK".check_changelog.pyandcheck_shared_blocks.pypass.Generated by Claude Code