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

Allow configuring brightness settings per display #21

Closed
xPaw opened this issue Mar 10, 2022 · 6 comments
Closed

Allow configuring brightness settings per display #21

xPaw opened this issue Mar 10, 2022 · 6 comments

Comments

@xPaw
Copy link

xPaw commented Mar 10, 2022

This is a simple and effective app!

I have two different displays, and as a result they require different brightness settings.

I think starting simple should be enough (get devices and display brightness setting for each of them, and store them in config by their device id). Might need to take care of being able to disable per device (or at least support the case when low/high brightness is the same value).

Per monitor/device brightness settings is mentioned under planned features, just making an issue to provide a use case.

@mruac
Copy link

mruac commented May 7, 2023

+1 for this, or being able to set minimum brightness per monitor when adjusting for sunset/sunrise brightnesses so that the brightness can stay consistent during the times its not adjusting.

@cristipopescu1
Copy link

+1 , perhaps the option to turn the app off for specific displays might also be handy

@alihassantahir
Copy link

+1 , perhaps the option to turn the app off for specific displays might also be handy

How about you disable DDC for that monitor? The SSB will not work for that monitor then.

@jacob-pro jacob-pro mentioned this issue Jan 10, 2024
@jacob-pro
Copy link
Owner

jacob-pro commented Jan 11, 2024

There is now support for this in the latest release, although you will need to configure it manually.

You will need to edit the config file: %LOCALAPPDATA\solar-screen-brightness\config.json

You can set monitor overrides like this:

{
  "brightness_day": 80,
  "brightness_night": 60,
  "transition_mins": 40,
  "location": {
    "latitude": 51.50745,
    "longitude": -0.12777
  },
  "overrides": [
    {
      "pattern": "CH89_C34H89xWJ (DP)",
      "key": "device_description",
      "brightness": {
        "brightness_day": 100,
        "brightness_night": 70
      }
    }
  ]
}

You can also disable brightness for that device:

    {
      "pattern": "CH89_C34H89xWJ (DP)",
      "key": "device_description",
      "brightness": null
    }

The keys and patterns can be found by calling: ssb-cli.exe --once

{
  "monitors": [
    {
      "device_name": "\\\\.\\DISPLAY1\\Monitor0",
      "properties": {
        "device_description": "CH89_C34H89xWJ (DP)",
        "device_key": "\\Registry\\Machine\\System\\CurrentControlSet\\Control\\Class\\{4d36e96e-e325-11ce-bfc1-08002be10318}\\0006",
        "device_name": "\\\\.\\DISPLAY1\\Monitor0",
        "device_path": "\\\\?\\DISPLAY#SAM0E25#5&40a2469&0&UID4354#{e6f07b5f-ee97-4a90-b076-33f57bf4eaa7}"
      },
      "brightness": null,
      "error": null
    },

key can be any of the listed device properties

pattern is a match against the value for that key, it can include * to indicate a wildcard match.

I will be working on a GUI for this soon to make it more friendly.

Please let me know if this is suitable for you?

@xPaw
Copy link
Author

xPaw commented Jan 11, 2024

Sounds reasonable.

@jacob-pro
Copy link
Owner

This is now available in the 2.1.0 release: https://github.com/jacob-pro/solar-screen-brightness/releases/tag/2.1.0

image

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

5 participants