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

RGB Underglow - minimum brightness for breathing animation #1689

Closed
drashna opened this issue Sep 7, 2017 · 5 comments
Closed

RGB Underglow - minimum brightness for breathing animation #1689

drashna opened this issue Sep 7, 2017 · 5 comments

Comments

@drashna
Copy link
Member

drashna commented Sep 7, 2017

As title. It would be nice to be able to set a minimum value (and possibly max) for the breathing animation so that it doesn't go completely dark.

And handle it like the RGBLED_BREATHING_INTERVALS, and similar values, so they could be defined in your keymap.

skullydazed added a commit that referenced this issue Sep 7, 2017
…posing defines to control the shape of the curve.
@skullydazed
Copy link
Member

This seems like a good idea to me. I took a stab at it with #1692, how's that look to you?

@drashna
Copy link
Member Author

drashna commented Sep 7, 2017

Well, it looks like #1692 still allows it to go dark, but the overall animation for it looks better!

@skullydazed
Copy link
Member

Give #define RGBLIGHT_EFFECT_BREATHE_CENTER 1 a shot.

@skullydazed
Copy link
Member

For reference, here is how RGBLIGHT_EFFECT_BREATHE_CENTER affects the breathing. The first image is the default setting of 1.85, which as you can see puts the lowest brightness at about 30/255, or ~12% (which may be below the threshold of visibility for your ws2812's.)

screen shot 2017-09-07 at 9 00 42 am

This second graph demonstrates how it looks when RGBLIGHT_EFFECT_BREATHE_CENTER is set to 1. As you can see the entire curve moves up, so the lowest brightness is 70/255, or ~27%, which should be visible on almost any board.

screen shot 2017-09-07 at 9 00 48 am

If you have OS X you can open
led_breathing.gcx.zip in grapher to play with the equation and get the curve to look how you want. You can from the images where to change RGBLIGHT_EFFECT_BREATHE_CENTER, but if you want to change RGBLIGHT_EFFECT_BREATHE_MAX too the correct number is the 255 on the right. We always need x/255 to get the curve to fit our available 256 brightness levels.

@drashna
Copy link
Member Author

drashna commented Sep 7, 2017

Perfect! Thanks!

@drashna drashna closed this as completed Sep 7, 2017
skullydazed added a commit that referenced this issue Oct 17, 2017
…posing defines to control the shape of the curve.
skullydazed added a commit that referenced this issue Oct 17, 2017
* Address #1689 by using a formula to define the breathing curve and exposing defines to control the shape of the curve.

* Tweak the behavior of breathing for clueboard
kgwong pushed a commit to kgwong/qmk_firmware that referenced this issue Nov 26, 2017
…k#1692)

* Address qmk#1689 by using a formula to define the breathing curve and exposing defines to control the shape of the curve.

* Tweak the behavior of breathing for clueboard
LovesTha pushed a commit to LovesTha/qmk_firmware that referenced this issue Jul 24, 2018
…k#1692)

* Address qmk#1689 by using a formula to define the breathing curve and exposing defines to control the shape of the curve.

* Tweak the behavior of breathing for clueboard
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

2 participants