Review of "Dynamic Brightness" version 3

Details Page Preview

A GNOME Shell extension that dynamically adjusts hardware screen brightness based on the active screen content luminance. Designed to provide eye comfort, especially when switching between light and dark content on laptop screens. Instead of capturing entire screen, this extension collects sample pixels to calculate your screen's luminance. This provides less cpu usage and longer battery life. While this extension provides a convenient, user-space solution for dynamic brightness on GNOME, it comes with certain architectural trade-offs: 1) Resource Usage: Because the extension periodically collects sample pixels and analyzes screen content via JavaScript (Shell.Screenshot) at set intervals, it introduces a continuous, minor CPU overhead. 2)Battery Impact: Frequent sampling and pixel data processing can impact battery life more than a native, hardware-level daemon would. Looking for a Battery-Efficient Alternative? If you are running Wayland and require a truly deep, native, and power-efficient solution for content-aware screen brightness (without the overhead of a GNOME extension), I highly recommend checking out wluma. It operates as an external, low-level daemon specifically optimized for performance and battery conservation. UPDATE: Screen brightness handling interface changed after Gnome 49, this version of extension supports both old and new interface

Extension Homepage
https://github.com/mnural/dynamic-brightness-gnome

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

Shexli (experimental)

Shexli did not find any issues.

All Versions

Version Status
6 Active
5 Inactive
4 Rejected
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
1. Please use `connectObject()` and `disconnectObject()` so it is easier to track for cleanup. 2. Remove line 173-174 `extension.js`: [Extension Best Practices Guidelines](https://gjs.guide/extensions/review-guidelines/best-practices.html) 3. Timeout should be removed before creating a new one (line 194 `extension.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) Didn't review. Please remove the version from import and follow our guidelines in [Extension Best Practices Guidelines](https://gjs.guide/extensions/review-guidelines/best-practices.html) first. Please avoid sending large diff. It can wait even more than a month since the review queue is very busy. If you need any help with your extension you can ask us on: - [GNOME Extensions Matrix Channel](https://matrix.to/#/#extensions:gnome.org) - IRC Bridge: irc://irc.gimpnet.org/shell-extensions
JustPerfection posted a review
Sorry for mentioning the large diff and version import. That was from the other notes I've had. That's not related to this extension.