enhancement: Adds IntelliTect celebration pattern - #6
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9d2d5377c9
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| } | ||
| break; | ||
|
|
||
| case 1: |
There was a problem hiding this comment.
Scope the switch cases to keep the sketch buildable
When the ESP8266 sketch is compiled as C++, this case 1 label jumps past the initialized center and ringWidth variables declared under case 0, which GCC rejects with “jump to case label crosses initialization.” Enclose each case body in braces (as the earlier implementation did) or move those declarations outside the switch; otherwise the firmware cannot be built.
Useful? React with 👍 / 👎.
Wokwi Link
Introduces a new
IntelliTectPatternfor celebration displays. This pattern features expanding rings of IntelliTect blue, white, and black emanating from the center of the LED strips. After the rings fully expand, the pattern transitions to a sustained pulse of solid IntelliTect blue across all strips.Example video shows full run along with consecutive successful hit resets:
IntelliTect.Celebration.Pattern.Example.mp4