Skip to content
This repository was archived by the owner on Apr 21, 2025. It is now read-only.

Add support for K70 Rapidfire (non-RGB) 0x1b3a#87

Merged
tatokis merged 4 commits into
mattanger:testingfrom
hevanaa:testing
Feb 26, 2017
Merged

Add support for K70 Rapidfire (non-RGB) 0x1b3a#87
tatokis merged 4 commits into
mattanger:testingfrom
hevanaa:testing

Conversation

@hevanaa

@hevanaa hevanaa commented Feb 26, 2017

Copy link
Copy Markdown
Contributor

This is now done against testing branch. I've also added the KB in question and the K70 LUX non-RGB to the IS_MONOCHROME macro. Hopefully this is correct.

@hevanaa

hevanaa commented Feb 26, 2017

Copy link
Copy Markdown
Contributor Author

Hmm, something went wrong there.

@hevanaa

hevanaa commented Feb 26, 2017

Copy link
Copy Markdown
Contributor Author

OK, think I solved the merge conflict (testing branch had changed after I forked it).

@tatokis

tatokis commented Feb 26, 2017

Copy link
Copy Markdown
Collaborator

From my understandint, the difference between a device being (!IS_RGB) or (IS_RGB && IS_MONOCHROME), is that the latter, while leveraging the RGB protocol, can allow for per-key lighting.
From Corsair's site I can see that both of these keyboard do support it, so we should be okay. Just need someone to test it.

Looks OK to me.

@hevanaa

hevanaa commented Feb 26, 2017

Copy link
Copy Markdown
Contributor Author

I've compiled and tested with my K70 LUX non-RGB. The Rainbow demo stopped working. The Trippy demo still worked. I guess the IS_MONOCHROME is no good then?

@hevanaa

hevanaa commented Feb 26, 2017

Copy link
Copy Markdown
Contributor Author

Couldn't get it to work at all with !IS_RGB. I'm removing them both from IS_MONOCHROME for now.

@hevanaa

hevanaa commented Feb 26, 2017

Copy link
Copy Markdown
Contributor Author

There must be some difference with the Strafe model (that has the IS_MONOCHROME macro) and at least this K70 LUX (non-RGB). Most things work in the GUI, except that pressing the Change Color button freezes the interface, but this happens both with if it is in IS_MONOCHROME and not (and it has always been like that with this model).

@tatokis

tatokis commented Feb 26, 2017

Copy link
Copy Markdown
Collaborator

pressing the Change Color button freezes the interface That's a Qt bug, we have info about it in the readme.

@ghost

ghost commented Feb 26, 2017

Copy link
Copy Markdown

Cross-reference to #86.

Works for @koanya who owns such keyboard. See #55 (comment).

@tatokis

tatokis commented Feb 26, 2017

Copy link
Copy Markdown
Collaborator

My guess is these keyboards discard the colour packets, so it should be safe to add them for now.

@tatokis tatokis merged commit 7e0ebab into mattanger:testing Feb 26, 2017
tatokis added a commit that referenced this pull request Feb 26, 2017
@hevanaa hevanaa deleted the testing branch March 1, 2017 17:48
frickler24 added a commit to frickler24/ckb-next that referenced this pull request Mar 1, 2017
See issue mattanger#87. If the KB is noted as a monochrome, the brightness
of the keyboard does not change if a color profile is loaded.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants