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

Add brightness #199

Closed
wants to merge 39 commits into from
Closed

Add brightness #199

wants to merge 39 commits into from

Conversation

hweeks
Copy link
Contributor

@hweeks hweeks commented Nov 26, 2020

No description provided.

hammywhammy and others added 30 commits November 20, 2020 07:12
feat: typescript, linting
feat: update release process
# [0.13.0](v0.12.0...v0.13.0) (2020-11-21)

### Features

* typescript, linting ([510644e](510644e))
* update release process ([eb21cf1](eb21cf1))
This moves all files to typescript, creates and deploys types, and makes a strip facotry instead of
a base class, while retaining all original tests

BREAKING CHANGE: strip class becomes a factory, private api changes
feat(core): move to ts completly, strip factory
# [1.0.0](v0.13.0...v1.0.0) (2020-11-22)

### Features

* **core:** move to ts completly, strip factory ([fc6e016](fc6e016))

### BREAKING CHANGES

* **core:** strip class becomes a factory, private api changes
feat: pixel breakdown to match strip
# [1.1.0](v1.0.0...v1.1.0) (2020-11-22)

### Features

* pixel breakdown to match strip ([137a623](137a623))
hammywhammy and others added 9 commits November 23, 2020 16:33
You can specify a maximum value for channels as well as a separate gamma per channel for more
control over LED output.
You can specify a maximum value for channels as well as a separate gamma per channel for more
control over LED output.
# [1.2.0](v1.1.0...v1.2.0) (2020-11-26)

### Bug Fixes

* allow color conversion based on channel maximums ([4f1d542](4f1d542))

### Features

* **color:** allow color conversion based on channel maximums ([2bd4264](2bd4264))
* add base config for function ([ac5bac9](ac5bac9))
@hweeks hweeks closed this Nov 26, 2020
@hweeks
Copy link
Contributor Author

hweeks commented Nov 26, 2020

oops

@ScreamZ
Copy link

ScreamZ commented Jun 16, 2023

@hweeks Why is this not merged ? @ajfisher ?

Any way to adjust brightness ?

@ajfisher
Copy link
Owner

@ScreamZ it's because it fundamentally changes a lot of assumptions around the repo and makes a lot of shifts to type script which wasn't really intended. As such this went into a little dead end.

Noirmally the way I adjust brightness on a standard RGB strip is simply apply a multiplier backwards on the values - eg divide by half etc. It's not perfect but that works. Alternatively, I drop the voltage on the strip a little as most RGB strips will happily run at 3.3v instead of 5 and so long as you supply >2.5v the logic circuit will work okay.

If you want to cherry pick the brightness parts out and make a PR then that would be much appreciated.

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.

6 participants