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

Libra2 Frontlight toggle doesn't turn off light with some values. #9674

Closed
mcdenko opened this issue Oct 23, 2022 · 5 comments · Fixed by #9676
Closed

Libra2 Frontlight toggle doesn't turn off light with some values. #9674

mcdenko opened this issue Oct 23, 2022 · 5 comments · Fixed by #9676
Milestone

Comments

@mcdenko
Copy link

mcdenko commented Oct 23, 2022

  • KOReader version: v2022.10
  • Device: Kobo Libra 2

Issue

When Frontlight value is set to 6,7,12, 14,23,24,28... (I didn't check out every value) and when I tap left bottom corner (Frontlight Toggle), the light is not turning off completely. Even if brightness value is set to 0. The same is when I use Menu/Settings/Frontlight and tap Toggle button. Any other values I use 1..5, 8..11, etc.. Work as It should.

One thing else. If I close my Kobo cover book (with dimmed frontlight), and then (after few seconds) I open it again frontlight is turned off complettely as it should!

Sorry for my english. I just want to help improving this great reading application.

Steps to reproduce

1 Set frontlight brightness to 5 or 6 or 12... on Kobo Libra 2
2 Tap left down corner
3 Frontlight is dimmed but not turned off complettely

Here is a part i have found on crash.log

error in subprocess: ./ffi/kobolight.lua:28: Wrong brightness value given!
stack traceback:
[C]: in function 'assert'
./ffi/kobolight.lua:28: in function 'setBrightness'
frontend/device/sysfs_light.lua:78: in function 'setNaturalBrightness'
frontend/device/sysfs_light.lua:41: in function 'setBrightness'
frontend/device/kobo/powerd.lua:248: in function 'setIntensityHW'
frontend/device/kobo/powerd.lua:306: in function 'func'
./ffi/util.lua:348: in function <./ffi/util.lua:319>
[C]: in function 'xpcall'
./ffi/util.lua:319: in function 'runInSubProcess'
frontend/device/kobo/powerd.lua:304: in function 'turnOffFrontlightHW'
frontend/device/generic/powerd.lua:109: in function 'toggleFrontlight'
...
frontend/ui/widget/container/widgetcontainer.lua:100: in function 'handleEvent'
frontend/ui/widget/container/widgetcontainer.lua:82: in function 'propagateEvent'
frontend/ui/widget/container/widgetcontainer.lua:100: in function 'handleEvent'
frontend/ui/uimanager.lua:788: in function 'sendEvent'
frontend/ui/uimanager.lua:52: in function 'default'
frontend/ui/uimanager.lua:1313: in function 'handleInputEvent'
frontend/ui/uimanager.lua:1407: in function 'handleInput'
frontend/ui/uimanager.lua:1456: in function 'run'
./reader.lua:302: in main chunk
[C]: at 0x000140bd
error in subprocess: ./ffi/kobolight.lua:28: Wrong brightness value given!
stack traceback:
[C]: in function 'assert'
./ffi/kobolight.lua:28: in function 'setBrightness'
frontend/device/sysfs_light.lua:78: in function 'setNaturalBrightness'
frontend/device/sysfs_light.lua:41: in function 'setBrightness'
frontend/device/kobo/powerd.lua:248: in function 'setIntensityHW'
frontend/device/kobo/powerd.lua:306: in function 'func'
./ffi/util.lua:348: in function <./ffi/util.lua:319>
[C]: in function 'xpcall'
./ffi/util.lua:319: in function 'runInSubProcess'
frontend/device/kobo/powerd.lua:304: in function 'turnOffFrontlightHW'
frontend/device/generic/powerd.lua:109: in function 'toggleFrontlight'
...
frontend/ui/widget/container/widgetcontainer.lua:100: in function 'handleEvent'
frontend/ui/widget/container/widgetcontainer.lua:82: in function 'propagateEvent'
frontend/ui/widget/container/widgetcontainer.lua:100: in function 'handleEvent'
frontend/ui/uimanager.lua:788: in function 'sendEvent'
frontend/ui/uimanager.lua:52: in function 'default'
frontend/ui/uimanager.lua:1313: in function 'handleInputEvent'
frontend/ui/uimanager.lua:1407: in function 'handleInput'
frontend/ui/uimanager.lua:1456: in function 'run'
./reader.lua:302: in main chunk
[C]: at 0x000140bd

regards

@Frenzie
Copy link
Member

Frenzie commented Oct 23, 2022

@zwim Is #9675 a fix for this?

@zwim
Copy link
Contributor

zwim commented Oct 23, 2022

No #9675 is only related to the AutoDim plugin.

@mcdenko Just to rule out that AutoDim has anything to do with your issue. Please disable it (gear-screen-autodim) and see if the issue consists.

@mcdenko
Copy link
Author

mcdenko commented Oct 23, 2022

I have AutoDim plugin disabled.

@mcdenko mcdenko changed the title Libra2 Frontlight togggle doesn't turn off light with some values. Libra2 Frontlight toggle doesn't turn off light with some values. Oct 23, 2022
@NiLuJe
Copy link
Member

NiLuJe commented Oct 23, 2022

Oh, hey, just encountered this myself while looking into some other stuff, lol.

c.f., #9609 (comment)

@mcdenko
Copy link
Author

mcdenko commented Oct 23, 2022

Oh, hey, just encountered this myself while looking into some other stuff, lol.

c.f., #9609 (comment)

:)))
Great! So it's almost done! :))Thank You for your reply! :)

@Frenzie Frenzie added this to the 2022.11 milestone Oct 25, 2022
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

Successfully merging a pull request may close this issue.

4 participants