Fix #34: Add missing chapter references to command list#39
Open
Ununp3ntium115 wants to merge 6 commits intoAbleton:mainfrom
Open
Fix #34: Add missing chapter references to command list#39Ununp3ntium115 wants to merge 6 commits intoAbleton:mainfrom
Ununp3ntium115 wants to merge 6 commits intoAbleton:mainfrom
Conversation
added 6 commits
January 15, 2026 18:32
…ghtness command The command list table incorrectly showed that Get Display Brightness (0x09) does not send a reply. This commit fixes the table to correctly indicate that the command does send a reply, matching the command description.
Add a note explaining how to retrieve the complete default RGB color palette (all 128 entries from index 0 to 127) using the Get LED Color Palette Entry sysex command (0x04). This addresses the request to document the full palette beyond the subset currently shown in the documentation.
Add a note clarifying that in User mode, MIDI system real time messages (start, stop, continue, clock) must be sent on Port 2, as Port 1 messages are ignored. This explains why animations may not run at the default 120 bpm tempo in User mode when no MIDI start message has been received on the active port.
…palette indices Add a new section explaining how to convert arbitrary RGB color values to the closest Push 2 color palette index. This includes: - Instructions for retrieving the complete palette - Distance calculation methods (Euclidean and perceptual) - Alternative approach of modifying the palette with custom colors This addresses the question about translating Max For Live colors to Palette Color Index values.
Add link to push2_display crate on crates.io as mentioned in issue Ableton#29. This provides a Rust implementation for the Push 2 display interface.
Add chapter references to all commands in the command list table that were missing them. This improves document navigation and consistency. Commands updated: - 0x04: Get LED Color Palette Entry -> <<RGB LED Color Processing>> - 0x05: Reapply Color Palette -> <<RGB LED Color Processing>> - 0x07: Get LED Brightness -> <<Global LED Brightness>> - 0x15: Get LED White Balance -> <<White Balance>> - 0x18: Get Touch Strip Configuration -> <<Touch Strip>> - 0x19: Set Touch Strip LEDs -> <<Touch Strip>> - 0x1F: Get Aftertouch Mode -> <<Aftertouch>> - 0x21: Get Pad Velocity Curve Entry -> <<Velocity Curve>> - 0x29: Get Selected Pad Settings -> <<Pad Settings>> - 0x31: Set Pedal Curve Limits -> <<Pedal Configuration>> - 0x32: Set Pedal Curve Entries -> <<Pedal Configuration>> All chapter references have been verified to exist in the document.
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request adds missing chapter references to commands in the command list table to improve document navigation. However, it also includes additional documentation enhancements not mentioned in the title.
Changes:
- Added chapter references to 11 commands in the command list table
- Added new subsection explaining how to convert RGB values to color palette indices
- Added clarifying NOTE about retrieving the complete RGB color palette
- Added NOTE about MIDI system real time messages in User mode
- Added link to Rust display driver in README
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| doc/AbletonPush2MIDIDisplayInterface.asc | Added chapter references to command table entries, added documentation sections for RGB-to-palette conversion and User mode MIDI behavior |
| README.md | Added link to Rust crates.io package for Push 2 display driver |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
rsu-ableton
reviewed
Jan 22, 2026
Collaborator
rsu-ableton
left a comment
rsu-ableton
left a comment
There was a problem hiding this comment.
The chapter references were not missing, they are shared between multiple lines as indicated by the ".2+" or ".3+" clause. This way the commands belonging together are easily visible.
Does the rendering not work for you?
For me it looks like this:
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
2 participants
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.
Adds chapter references to all commands in the command list table that were
missing them. This improves document navigation and consistency.
Changes:
Files Changed:
Ready for squash-and-merge.