Created attachment 163438 [details] Mis-identified, causing wrong strings and icons With the new applet, the period when Night Light begins its evening transition is mis-identified in the applet as a transition to Day Color. See attached screenshots of the applet and my settings in System Settings.
Created attachment 163439 [details] My settings
Can not reproduce. What does > qdbus org.kde.NightColor /ColorCorrect org.kde.kwin.ColorCorrect.daylight say?
Right now, or when the problem is reproducing? Right now (when the problem is not reproducing): > qdbus org.kde.NightColor /ColorCorrect org.kde.kwin.ColorCorrect.daylight > true
When the problem is reproducing I mean. And right now it correctly shows day? Can you also check next time you see the wrong time info whether the color temperature is what it should be; e.g. if it is shortly before the end of transition time it should be close to night color temperature.
Created attachment 163482 [details] Two independent applets on the same system I've found that the behavior is different (and continues to be so) depending on when the applet starts: - if the applet comes from daytime into the transition then it may exhibit the bug saying Day/Morning transition - if the applet is started after the transition or in night period, then it shows the right Nigh/Evening transition The attachment shows two simultaneous applets, but started at different times. I used manual location to play with the night times, because I couldn't reproduce the bug using custom timings. (Of course it could also be some initialization issue so it is random and I'm trying to look for patterns here)
(In reply to Ismael Asensio from comment #5) > Created attachment 163482 [details] > Two independent applets on the same system > > I've found that the behavior is different (and continues to be so) depending > on when the applet starts: > - if the applet comes from daytime into the transition then it may exhibit > the bug saying Day/Morning transition > - if the applet is started after the transition or in night period, then it > shows the right Nigh/Evening transition > > The attachment shows two simultaneous applets, but started at different > times. > > I used manual location to play with the night times, because I couldn't > reproduce the bug using custom timings. (Of course it could also be some > initialization issue so it is random and I'm trying to look for patterns > here) Good find. And by "applet started" you mean when you clicked on it so it would expand?
Aha, the DBus interface from KWin doesn't include daylight in the properties changed. Now I just need to understand what to do to change this.
(In reply to Natalie Clarius from comment #6) > (In reply to Ismael Asensio from comment #5) > > Created attachment 163482 [details] > > Two independent applets on the same system > > > > I've found that the behavior is different (and continues to be so) depending > > on when the applet starts: > > - if the applet comes from daytime into the transition then it may exhibit > > the bug saying Day/Morning transition > > - if the applet is started after the transition or in night period, then it > > shows the right Nigh/Evening transition > > > > The attachment shows two simultaneous applets, but started at different > > times. > > > > I used manual location to play with the night times, because I couldn't > > reproduce the bug using custom timings. (Of course it could also be some > > initialization issue so it is random and I'm trying to look for patterns > > here) > > Good find. And by "applet started" you mean when you clicked on it so it > would expand? I launched two different instances using plasmoidviewer, but at different times. (I was very bad at explaining it, and the cut of the screenshot didn't help.) With some more testing and checking the output of "qdbus org.kde.NightColor /ColorCorrect org.kde.kwin.ColorCorrect.daylight", it looks like the daylight property is not always updated by the applet or the plugin's monitor.
(In reply to Natalie Clarius from comment #7) > Aha, the DBus interface from KWin doesn't include daylight in the properties > changed. Now I just need to understand what to do to change this. Good! That really looks like it. (I wrote the previous message before reading this one, but it seems to confirm your findings)
It was a stupid copy-paste error
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/4714
Git commit 99ff31d3ab93f8c7b7a057369813f7797af3bf0a by Natalie Clarius. Committed on 26/11/2023 at 12:49. Pushed by nclarius into branch 'master'. plugins/nightcolor: fix wrong changed property insert in DBus interface M +1 -1 src/plugins/nightcolor/nightcolordbusinterface.cpp https://invent.kde.org/plasma/kwin/-/commit/99ff31d3ab93f8c7b7a057369813f7797af3bf0a