Add spectrograph support for SteelSeries Apex M800 #52
Conversation
|
Nice pull request, code looks clean and easy to merge! One thing I want to verify before I merge it in is that it properly returns FALSE and stops executing the thread if the Apex M800 is not detected. Thanks for adding this! |
|
Build was clean, didn't affect operation of existing devices. Consider it merged! |
|
Great! Thanks! |
|
Do you know what keyboard layout you used to implent this? When doing the protocol from scratch you need to account for other region's keys. Finding those out is usually trial and error after you find someone to test who has a different region keyboard. A reddit user tested v2.01 and said two keys were not lighting on his UK layout keyboard. I'd like to figure out what the ID of those keys are to support that layout. |
|
It was a US layout M800. I should be able to add support for UK layout. |
|
Great! One other change that I was going to make tonight is to start at Y=2 rather than Y=0. The first two rows are reserved, row 0 is for the linear bar graph visualizer used on LED strip style devices and row 1 is for single color mode devices like mouse logos and devices with only one RGB LED. The Fn row was showing the horizontal bar graph which is not supposed to happen. |
|
I just created a new pull request for the UK layout, #55 |
This code adds spectrograph support for the SteelSeries Apex M800 keyboard