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

[Feature request]: Minimum brightness setting independent of light sensor #247

Open
FabianMPunkt opened this issue Nov 25, 2024 · 3 comments
Assignees
Labels
feature request keep don't stale this issue

Comments

@FabianMPunkt
Copy link

Is your feature request related to a problem? Please describe

Hey, i recently switched from Prismatik to Luficerin, as i wanted to use a software that is more future-proof. While Luciferin is a more robust solution, i still feel like there are quite a few features missing. One such feature is a simple "Minimum brightness" setting. I like to have just a bit of ambient light behind my screen when it is pitch black, additionally, in very dark scenes, my LEDs constantly switch between being off, and being very dim, which often results in very bad flickering.

My setup:
ESP32-C3 Supermini via WiFi
126x WS2812B RGB
Currently running on Windows 11
image

Describe the solution you'd like

Prismatik adresses this with a simple threshold setting:
Screenshot 2024-11-12 183455
Personally, i only care about the "Minumum luminocity level" - these are my old Prismatik settings.

Describe alternatives you've considered

I know Luciferin has a similar setting but that only works when combined with a light sensor. I currently don't have on installed, but i could botch one on there ;)
I'd love to contribute to this project myself but unfortunately i have exactly zero knowledge about java and all that stuff.

@sblantipodi
Copy link
Owner

Hi there,
nice to know that you jumped on Luciferin :)

That feature is already present.
https://github.com/sblantipodi/firefly_luciferin/wiki/Eye-care-and-night-mode

night_mode~2.jpg

Just toggle the eye care toggle, that one doesn't require LDR.
The minimum brightness should be ok.

Please let me know if it's ok for you.

@FabianMPunkt
Copy link
Author

Hey thanks, I did not realize that was already a thing :D
It would be great if there was a slider to customize the threshold value for this feature. That would also make it's existance more obvious. I always assumed that this checkbox was there to toggle the night-mode.

Also, on that note about night mode..
Are there any plans to support the native Night light feature in windows?
Prismatik has this checkbox called "Apply screen color modification"
image
This basically mirrors the night light effect and applies the same orange tint to the LEDs.
Would something like this even be possible using DDUPL?

@sblantipodi sblantipodi added the keep don't stale this issue label Nov 27, 2024
@sblantipodi
Copy link
Owner

I agree that the toggle is not that clear :)

regarding the windows feature yes it's possible.
I have other features on the roadmap but I add this for later... 👍

thanks for the suggestion!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request keep don't stale this issue
Projects
None yet
Development

No branches or pull requests

2 participants