-
Notifications
You must be signed in to change notification settings - Fork 42
re-basing EEZ branch to latest main #283
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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
…hen "sync" is enabled. E.g. when Delay sync is disabled, the Midi param sets the time. When sync is enabled, the Midi param sets the time stamp value.
…ained together would only accept the first one. Devices like the Chocolate pedal can now e.g. enable modulation and select the modulation model from a single footswitch press.
…l pre-made controllers. More news soon. Only change is to enable serial Midi by default.
- Added 1.69" landscape build for collaboration with Pirate Midi
- set Pirate Midi Polar Plus to use 180 degree screen rotation by default - renamed Pirate Midi builds to match the product names
- fixed compatibility issue with Ampero Control preventing connection over Bluetooth (thanks to "jjh84" for debugging)
…der, appended CC, mixed CC and PC etc. Shared between all 3 Midi sources. Start of support for LilyGo T-Display-S3 board with 1.9" landscape LCD (currently not functional).
…ft/right swipe gestures for preset next/previous.
…uild for user testing.
- completed Waveshare 1.69" touch support with left/right swipe for preset changing - enabled pullup on Midi RX line, to help cases where pin is floating with nothing connected - moved some components from unmanaged to managed - more work on Lilygo T-Display S3 platform (not complete) (note: display doesn't work on ESP IDF 5.0.2, needs later) - increased task watchdog timeout due to issues when testing with newer ESP IDF versions - refactored display module to handle touch screens more generically
- added code to purge any rubbish Midi received during boot/init
Enabled support for USB Hubs in config (but Tonex not working with hubs yet) Initial testing on IDF V5.4.1 is progressing well but not complete.
- changed platforms with 2 MB PSRAM to no longer copy instructions to PSAM on boot, due to the PSRAM running out of heap space
- small tweak preparing for USB hub support
Updated to add more videos, links to articles, details of the Pirate Midi products, and a notice to code thieves who are violating the project license terms.
…waiting hardware. Shared UI with Lilygo T-Display S3.
Fixed a few minor errors
…onal yet. Also added touch swipe to 1.9" but also not complete.
…h 20 and 150 preset maximums.
… 4.3B, to help where screen protector is fitted - added message to web about enabling mDNS in router when station mode is selected - added UDP broadcast locator, to help with finding connected devices - added Windows app for above
…t A/B slot targeting - Add CC 120: Load Preset to Slot A (values 0-19) - Add CC 121: Load Preset to Slot B (values 0-19) - Both commands bypass global slot configuration - Both commands load without switching active slot (selectSlot=0) - Maintain full backward compatibility with CC 127 - Add proper validation and error logging - Update MIDI commands documentation Resolves independent A/B preset loading for HX Stomp workflows
…swipe preset changing.)
… showing the device IP address.
TonexOneController: add ci to build the system
… add CI to project. Specifically, renaming the 43devonly config to remove the space in name, and changing the order of the sdkconfig defaults. Also allocated two midi CC values for putting presets into particular slots.
Also updated to ESP IDF 5.5.1.
# Conflicts: # source/main/midi_helper.c
…ral info that minimises the chance of accidental parameter changing.
…t targeting - Add CC 120 (Load Preset to Slot A) and CC 121 (Load Preset to Slot B) - Implement thread-safe architecture using message queues instead of direct USB calls - Add USB_COMMAND_LOAD_PRESET_TO_SLOT_A/B commands to enum - Add usb_load_preset_to_slot_a/b() queue functions in usb_comms.c - Add command handlers in usb_tonex_one_handle() switch statement - Update MIDI handlers to use new thread-safe functions - Allows HX Stomp to independently load presets into both slots without forcing mode changes - Preserves TONEX ONE's native A/B toggle functionality and tuner access Addresses PR feedback regarding thread safety - replaced direct USB function calls from MIDI thread with proper message queue pattern as suggested by reviewer.
Hid the BPM indicator on 3.5B as it takes a fair bit of CPU usage to redraw the entire screen each time.
feat: add MIDI CC 120/121 for independent A/B slot preset loading
…via Managed Components. Also some tweaks to the touch init sequence on Lilygo T-Display S3, due to user issues.
…utton. - Added support for Midi thru
- made Midi passthru an sdkconfig item, and disabled it for the Waveshare 4.3B builds, as the RS485 transceiver gets in the way
…, for JC3248 and Waveshare 3.5" platforms. Also fixed issue where the keyboard overlapped the text entry box.
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.
No description provided.