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

Set Default Brightness Level #67

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rtepowers
Copy link

The NeoPixels are incredibly bright. This change tones down the brightness for this particular example. As an added bonus, this commit also utilizes the 'setBrightness' method which helps demonstrate its functionality.

The NeoPixels are incredibly bright. This change tones down the brightness. As an added bonus, this commit also utilizes the 'setBrightness' method which helps demonstrate its functionality.
@rtepowers rtepowers mentioned this pull request Oct 29, 2015
@mscosti
Copy link

mscosti commented Oct 29, 2015

I was also thinking it could be interesting if we had a new animation function, possibly named 'pulse'. It would basically animate through the whole brightness range from 0 - 255 (or whatever looks best), for all red, then all green, then all blue.

Besides looking cool it might bring more attention to setBrightness when people see it actively being changed when they run the example the first time.

Definitely keep it defaulted to lower than 255 like you have already though.

Thoughts?

@UIOP456y
Copy link

Thank you for the brightness changes for the "strandtest". Works great,
give me a little insight into the coding, still have alot to learn.

Thanks again

On Thu, Oct 29, 2015 at 6:02 PM, R. Powers [email protected] wrote:

The NeoPixels are incredibly bright. This change tones down the brightness
for this particular example. As an added bonus, this commit also utilizes

the 'setBrightness' method which helps demonstrate its functionality.

You can view, comment on, or merge this pull request online at:

#67
Commit Summary

  • Set Default Brightness Level

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#67.

@tablatronix
Copy link
Contributor

👍

@ladyada
Copy link
Member

ladyada commented May 4, 2016

@tablatronix plz dont +1 issues, it fills my inbox and doesn't make me code faster ;) please use the "reactions" button on the issue if you want to 'vote' on something

@tablatronix
Copy link
Contributor

Word.
Reactions are not available on mobile web version, I forget about them. I think you can leave watch on but disable emails... somewhere.

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

Successfully merging this pull request may close these issues.

5 participants