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]: print more debug info #108

Open
Atemu opened this issue May 22, 2024 · 3 comments
Open

[Feature request]: print more debug info #108

Atemu opened this issue May 22, 2024 · 3 comments

Comments

@Atemu
Copy link

Atemu commented May 22, 2024

Please describe your feature request

In the debug log (RUST_LOG=debug) it currently only prints the initial config and new data points (lux class, content(?) luma, brightness) which I find a bit lacking.

For debugging, it'd be really useful to also see:

  1. Brightness change actions taken
    I had the case of GNOME's auto-brightness feature suddenly appearing and accidentally running two instances of wluma where brightness would change erratically. This would have been a lot easier to debug if the wluma instance I was looking at printed a line of debug info every time it actually took an action.
  2. Brightness value read/inferred from the sensor
    When setting up wluma, it's not obvious how to set up the lux "classes" (dim, normal, bright etc.) because you don't have a reference. You could start reading the device's output manually in addition to the debug log but this isn't necessarily trivially possible with webcam sources and even with iio sources it'd be really convenient to just have that info at that point in time.
@maximbaz
Copy link
Owner

hello! I'm on a phone now, but I'm pretty sure the trace log level is printing exactly the info you are looking for, could you give it a go?

@Atemu
Copy link
Author

Atemu commented May 22, 2024

Indeed it is. I mistakenly thought debug was the highest level.

For reference, it outputs a constat stream of logs of this pattern

[2024-05-22T13:48:56Z TRACE wluma::predictor::controller] Prediction: 627 (lux: normal, luma: 0)
[2024-05-22T13:48:56Z TRACE wluma::als::iio] ALS (iio): normal (8)
[2024-05-22T13:48:56Z TRACE wluma::als::iio] ALS (iio): normal (8)
[2024-05-22T13:48:56Z TRACE wluma::predictor::controller] Prediction: 627 (lux: normal, luma: 0)
[2024-05-22T13:48:56Z TRACE wluma::als::iio] ALS (iio): normal (8)
[2024-05-22T13:48:56Z TRACE wluma::als::iio] ALS (iio): normal (8)
[2024-05-22T13:48:56Z TRACE wluma::predictor::controller] Prediction: 627 (lux: normal, luma: 0)
[2024-05-22T13:48:56Z TRACE wluma::als::iio] ALS (iio): normal (8)
[2024-05-22T13:48:56Z TRACE wluma::als::iio] ALS (iio): normal (8)

While trace does print the brightness wluma gets from the sensors, it does not print the actions taken. This would still be useful to have.

@maximbaz
Copy link
Owner

maximbaz commented Jun 16, 2024

Hi again, sorry I forgot to reply, Prediction: 627 is the action being taken, wluma is writing the value 627 into your brightness raw file (with some smooth transition though).

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

2 participants