[GH-ISSUE #331] Brightness on another VCP code #262

New issue

Closed

opened 2026-05-05 05:38:37 -06:00 by gitea-mirror · 3 comments

gitea-mirror commented

2026-05-05 05:38:37 -06:00

Owner

Originally created by @Giacomo-Sanna on GitHub (Nov 22, 2020).
Original GitHub issue: https://github.com/MonitorControl/MonitorControl/issues/331

Checklist

I have searched for existing issues

Question
When I try to change the brightness of the monitor, the black level is changed instead.
After some researches, I discovered that my monitor isn’t following the last VESA Monitor Control Command Set standard.
Basically to change the monitor brightness you set inDDC.swift
case luminance = 0x10
but for my monitor I need to set it to 0x13.
I was wondering if I need to build the project with my changes, or I can edit something in the .dmg file.

Originally created by @Giacomo-Sanna on GitHub (Nov 22, 2020). Original GitHub issue: https://github.com/MonitorControl/MonitorControl/issues/331 **Checklist**  - [x] I have searched for existing issues **Question** When I try to change the brightness of the monitor, the black level is changed instead. After some researches, I discovered that my monitor isn’t following the last VESA Monitor Control Command Set standard. Basically to change the monitor brightness you set in`DDC.swift` `case luminance = 0x10` but for my monitor I need to set it to `0x13`. I was wondering if I need to build the project with my changes, or I can edit something in the .dmg file.

gitea-mirror closed this issue

2026-05-05 05:38:39 -06:00

gitea-mirror commented

2026-05-05 05:38:43 -06:00

Author

Owner

@JoniVR commented on GitHub (Nov 22, 2020):

Yes, you'd probably want to create a fork of this project, change the value to .backlightControlLegacy (0x13) and then build it yourself. I think we could also provide this as an advanced preferences option in the future, but that's probably a long way off unless you'd want to implement it in a PR.

@JoniVR commented on GitHub (Nov 22, 2020): Yes, you'd probably want to create a fork of this project, change the value to `.backlightControlLegacy` (0x13) and then build it yourself. I think we could also provide this as an advanced preferences option in the future, but that's probably a long way off unless you'd want to implement it in a PR.

gitea-mirror commented

2026-05-05 05:38:47 -06:00

Author

Owner

@Giacomo-Sanna commented on GitHub (Nov 22, 2020):

Thanks, I'll do it myself.
But in the future, if you will develop it, it would be great.

@Giacomo-Sanna commented on GitHub (Nov 22, 2020): Thanks, I'll do it myself. But in the future, if you will develop it, it would be great.

gitea-mirror commented

2026-05-05 05:38:49 -06:00

Author

Owner

@JoniVR commented on GitHub (Dec 5, 2020):

If you want it in, you or someone else will have to submit a Pull Request.

You can't expect maintainers to do all the work, that's not a sustainable approach for any of us as this is in no way something we do full-time.

I'm open to merging it as an Advanced preferences feature if anyone is willing to submit it. Closing for now, feel free to reopen or ping me to reopen 🙂

@JoniVR commented on GitHub (Dec 5, 2020): If you want it in, you or someone else will have to submit a Pull Request. You can't expect maintainers to do all the work, that's not a sustainable approach for any of us as this is in no way something we do full-time. I'm open to merging it as an Advanced preferences feature if anyone is willing to submit it. Closing for now, feel free to reopen or ping me to reopen 🙂