Fix boot, GPIO, tool-import, and OM budget regressions on v0.7.0-beta.1.#17
Open
kadders wants to merge 3 commits into
Open
Fix boot, GPIO, tool-import, and OM budget regressions on v0.7.0-beta.1.#17kadders wants to merge 3 commits into
kadders wants to merge 3 commits into
Conversation
- Fix OM global overflow (~8 KiB) that discarded DSF responses and caused DWC timeouts after boot and tool load - Fix Scylla GPIO crash when nxtBoardFanPins was a string or "none" - Fix silent board-pack skip on first M999 (bootstrap auto / non-empty sentinel) - Fix spindle and fan-pin declare-or-set aborts mid nxt-vars / user-vars - Fix Tool Library import disconnect from burst M4000 HTTP (scratch upload + one M98) - Fix Fusion import dropping high pockets by reserving only the probe slot - Replace nxtToolCache vector with nxtToolCacheIdx / nxtToolCacheZ - Defer nxtPinStates and nxtToolLife vectors until first use - Gate Custom A-axis globals behind nxt-custom-a.requested - Declare board-pack telemetry globals on use instead of always-on nulls - Strengthen OM budget checker (size estimate + known cliff bans) - Harden Scylla GPIO role defaults and fan-mode pin create (CSV membership, clearer locals) Co-authored-by: Cursor <cursoragent@cursor.com>
Replace the unused LED strip index control with a 1–256 count field persisted in nxt-user-vars.g, stop writing count from nxt-save-rgb.g, and re-apply user-vars after colour load so boot M950 keeps the configured length. Co-authored-by: Cursor <cursoragent@cursor.com>
Load MosAtc from nxt.g only when the feature flag and SD init macros are present (skip plugin-init dispatch for OM budget), expose the ATC toggle in Configuration, remap Scylla named outputs to aux0–2 → coolant → mist → relay with fan default aux0, and harden Custom homing travel/Z-clearance expressions for RRF. Co-authored-by: Cursor <cursoragent@cursor.com>
kadders
commented
Jul 18, 2026
| if { exists(param.I) && exists(global.nxtTouchProbeID) && global.nxtTouchProbeID != null } | ||
| if { param.I == global.nxtTouchProbeID } | ||
| set var.applyTouchDefl = true | ||
| if { var.applyTouchDefl && exists(global.nxtProbeDeflection) && global.nxtProbeDeflection != null } |
Contributor
Author
There was a problem hiding this comment.
Double check the use of these indexes, if we are referencing Z it should be index [2]
| | Air Blast | Any free named gpOut | Aux 2 | | ||
| | Mist Coolant | Prefer Mist pin | Mist (`A.7`) | | ||
| | Flood / Coolant | Prefer Coolant pin | Coolant (`C.4`) | | ||
| Configure GPIO outputs from **named board pins** (Mist, Coolant, Relay, Aux, …). On Scylla, the board pack creates ports and fills null role IDs in order **mist → coolant → aux0 → aux1 → aux2 → relay**. |
Contributor
Author
There was a problem hiding this comment.
This pin mapping is up for discussion in terms of order (mainly due to legacy config pin map)
Contributor
Author
There was a problem hiding this comment.
Note to drop this file going forward if no longer needed
Contributor
Author
There was a problem hiding this comment.
This will eventually need to be updated for a startup fan needing to be set in the future
Contributor
Author
There was a problem hiding this comment.
Flagging this file for removal if not needed going forward
Contributor
Author
There was a problem hiding this comment.
New file for defining GPIO things, let's make sure this is not duplicated with with the other gpio.g
Contributor
Author
There was a problem hiding this comment.
New config file for the uart settings if configured.
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
1 participant
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.
Fix boot, GPIO, tool-import, and OM budget regressions on v0.7.0-beta.1.