Bump amy submodule to 145e3d8f (revert of amy#804 + amy#820)#1099
Merged
Conversation
Picks up two fixes on amy main: - amy#822: revert of amy#804, which broke AMYboard boot entirely (StoreProhibited crash-loop in esp_clk_init before app startup; bisected and boot-verified on the HW CI bench) - amy#820: stop allocating the 512KB sysex copy-slot ring on Tulip, freeing enough SPIRAM that tulip.screenshot() works with WiFi up again (Tulip HW CI back to full PASS: audio 0.9992, screenshot pixel-diff 0.00) refdocs synced from the pinned commit (only the vendored-from SHA changed; amy docs content is unchanged by these commits). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
🔌 AMYboard PR previewEditor + flasher: https://amyboard-pr-1099.vercel.app/editor/ This preview bundles this PR's firmware — its flasher only flashes this build (not the release). Rebuilt on every push; removed when the PR closes. The hardware CI has been kicked off and should return within a few minutes, stand by! |
🌷 Tulip Web PR previewTulip Web: https://tulip-pr-1099.vercel.app/run/ Flash a Tulip to this build: on-device run Rebuilt on every push; removed when the PR closes. |
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.
Pins amy to current amy main (145e3d8), picking up two fixes:
esp_clk_initbefore app startup). Bisected and boot-verified on the HW CI bench: amy main+Skip saved env when loading from URL param #820 crash-loops, main+Skip saved env when loading from URL param #820+revert boots and enumerates USB-MIDI.tulip.screenshot()works with WiFi up again. Validated on the bench via the amy-PR HW CI chain: Tulip full PASS (audio 0.9992, screenshot pixel-diff 0.00).refdocs re-synced from the pinned commit (docs content unchanged, only the vendored-from SHA).
Supersedes #1094 (that pin is the boot-breaking commit and should be closed).
Known remaining HW CI issue (pre-existing, unrelated to this pin): AMYboard
hwci_basicscores 0.87 vs its 0.9 threshold since #1084 changed the kit samples — needs an--update-referencepass on the bench.🤖 Generated with Claude Code