Skip to content
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

Ergodox Infinity LCD light issue on the slave-half #1456

Closed
roded opened this issue Jul 2, 2017 · 9 comments
Closed

Ergodox Infinity LCD light issue on the slave-half #1456

roded opened this issue Jul 2, 2017 · 9 comments
Assignees

Comments

@roded
Copy link

roded commented Jul 2, 2017

Hi,
Using the 0.5.69 tag, I'm experiencing an issue with the LED lights on the half of the keyboard which is not connected to the computer (same behavior for both the left and right halves).
The LED display works and updates just fine but the lighting is silent.
I'm using the provided colemak keymap if that makes a difference.

@roded roded changed the title Ergodox Infinity LED issue on the slave-half Ergodox Infinity LED light issue on the slave-half Jul 2, 2017
@fredizzimo
Copy link
Contributor

fredizzimo commented Jul 3, 2017

You are not the first one to report this #1415

But it doesn't occur on all keyboards and I don't have LEDs installed myself to test this myself, so currently I have no idea what's wrong. But I will read the code carefully to see if there's some potential issue.

I'm closing this as a duplicate of #1415

@roded
Copy link
Author

roded commented Jul 3, 2017

Apologies, I mistyped LED instead of LCD. My issue is with the LCD backlights, not LED backlights.

@roded roded changed the title Ergodox Infinity LED light issue on the slave-half Ergodox Infinity LCD light issue on the slave-half Jul 3, 2017
@fredizzimo
Copy link
Contributor

Ah, I will re-open this then.

The underlying cause could still be the same, but since the behaviour is different, we should have two different issues.

@fredizzimo fredizzimo reopened this Jul 3, 2017
@fredizzimo fredizzimo self-assigned this Jul 4, 2017
@fredizzimo
Copy link
Contributor

Does anyone else experience this? Any additional information or help with debugging this issue is would greatly be appreciated.

@mhweaver
Copy link

mhweaver commented Jul 10, 2017

I've got the same issue - the LCD works on both halves, but the LCD backlight on the slave half doesn't seem to work with QMK. Not sure what extra info I can provide, other than verifying that more than one person has the problem, though...

Edit: I was messing around with visualizer.c and removed the is_serial_link_master() check and always set new_index and old_index with get_led_index_master(). The LCD backlights match, mimicking the behavior of the stock KLL firmware, which I'm fine with for now. get_led_index_slave() doesn't appear to want to give me a useful index.

@fredizzimo
Copy link
Contributor

Ah, after reading your edit, I now know what the issue is.

The behaviour is like that by design. The default visualizer tries to emulate the LEDs of the standard Ergodox, which has three LEDs, a red, a green and a blue one. It's a bit tricky to emulate that, with just two screens, so it works like this.

  • If one "LED" is on, then display that color and brightness on the left screen.
  • If two "LEDs" are on, display one color on one screen, and the other color on the other.
  • If three "LEDs" are on alternate the color on the second screen.

But that's the default visualizer, which is used, if you don't implement your own. It's only there to display somewhat meaningful information, on keymaps without a custom one.

The easiest way to make a custom visualizer, is to use simple_visualizer, check out the default keymap for an example. You can also implement a completely custom visualizer for more interesting effects.

I will soon have time to write documentation, so stay tuned for that.

@gabeplaysdrums
Copy link
Contributor

@mhweaver could you possibly share a topic branch with the code changes you made to get the backlight color showing up on both halves? That's the behavior I'd like but having trouble groking the changes needed.

@dudeofawesome
Copy link
Contributor

You can check out my ErgoDox layout, which links the LCD backlight color between halves. The line that does that is dudeofawesome/visualizer.c#L189.

@drashna
Copy link
Member

drashna commented Mar 25, 2018

"by design" and fix linked.

Closing.

@drashna drashna closed this as completed Mar 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants