Sunday, 2021-12-05

Mister_MagisterThaodan: since spiiroin is no longer with us [*] and since you touched mce too… would it be possible to merge this? https://github.com/spiiroin/mce/tree/cpu_govermor_revival i could really use that01:55
riniguspoetaster and piggz: catching up from last night. let's go for it then and set it as optional data. as it is human readable, I will probably even skip %if in my apps - it is useful info anyway07:26
riniguswouldn't consider it is as a hack myself. in many aspects, it is as simple for developers to provide such data as possible. no forms to fill, all is rather simple and possible to change together with the app source.07:28
rinigusI will summarize discussion in github and will try to see if I can plugin such parser into chum-gui07:29
poetasterrinigus: I agree that it is 1) readable, 2) easy to maintain while developing, 3) 0 overhead central maintenance09:44
poetasterrinigus: the only work is in the chum gui, which needs work one way of the other.09:44
poetasterrinigus: I'll start implementing it in my packages. And hopefully make some progress :)09:45
riniguspoetaster:: now that fields are not fixed yet. So, while we have the first proposed set, be ready to changes10:32
poetasterrinigus: sure thing, it was a future promise. I don't think I can help with code today. Family. But I'll check in the evening.10:34
ThaodanMister_Magister: I would try to highlight him or create an issue about that. But I'll forward hat internally.11:37
Mister_MagisterThaodan: thanks <311:38
ThaodanMister_Magister: For what did you want to use it?11:39
Mister_MagisterThaodan: UI is not very smooth on x2, its super laggy but if i set one core's frequency to minimum 1.2GHz then its waaaaaay smoother so i would like mce to set the min frequency on single core while ui is being used and while its idle11:40
ThaodanMister_Magister: that sounds like a good idea, maybe also help to preserve some power.11:41
Mister_Magisterglad that you think so11:42
Mister_Magisterwhole reason for this branch is because i asked him for that :D11:42
ThaodanHe was probably to busy and forgot about it.11:42
rinigusMister_Magister: we are using https://github.com/sailfishos-sony-tama/zgovernor on Tama for similar reasons. as I needed also to boost GPU, had to write a small daemon and didn't rely on that mce branch11:46
rinigusin the end, we are just adjusting GPU in the current config and let CPU work non-touched.11:46
rinigushere is the current config: https://github.com/sailfishos-sony-tama/droid-config-sony-tama-pie/blob/hybris-10/sparse/etc/zgovernor.ini11:47
Mister_Magisterchanging gpu doesn't change anything for me11:48
rinigusMister_Magister: have you tried to adjust GPU? note that in case of Tama, GPU had to be bumped ONLY when just lipstick was shown. as soon as we had an app displayed, all was fine11:48
Mister_Magisteryes11:48
Thaodanrinigus: Nice idea I hope we can convert that into an mce plugin11:48
rinigusfor CPU, same can be used. see https://github.com/sailfishos-sony-tama/zgovernor/blob/master/example/zgovernor.ini11:49
Mister_Magisteri will still probably just wait for mce branch11:49
Mister_Magisterbut thanks11:49
rinigusMister_Magister: sure. no problem - it just allows you to experiment now :)11:49
Thaodanthe mce branch does that too11:50
Thaodanjust rebase against master11:50
Mister_Magistermhm11:50
Mister_Magisterand since i did tests manually i just need to automate it11:50
rinigusThaodan: the problem with constant switch of CPU governors was bringing system bit unstable, if I remember correctly. so, was not much gain but more pain for Tama. but GPU bump is needed11:50
Mister_Magisterwas thinking of modifying govenor to prioritize/bump compositor workload but eh11:51
Mister_Magisterim not planning on changing govenor11:51
Mister_Magisterjust setting min frequency11:51
rinigusMister_Magister: that's probably better11:51
Mister_Magisteron single, 0, cpu as its always online11:51
Mister_Magisterother cpus will scale with workload but they will have normal min frequency of 30011:52
Mister_Magisterand since 0 is always online the composer thread will stay on 011:52
Mister_Magisterrinigus: if i wanted to workaround it without mce i would jsut write my own kernel module :P11:54
rinigusMister_Magister: kernel module that would follow DBus to change CPU/GPU settings according to the displayed windows. good luck!11:55
Mister_Magisternah11:55
Mister_Magisterkernel module that would change min frequency depending on display state11:55
Thaodanrinigus: Kdbus to rescue xD12:05
ThaodanPower Saving is a big WIP in SailfishOS IMHO12:06
rinigusThaodan: :) . well, I am happy about the solution I have for tama and it was rather simple to implement and test...12:07
rinigusThaodan: whats an issue with power saving on SFOS official devices? do you know for how long the battery lasts on xperia 10ii or 10? is it a problem?12:07
ThaodanThere's a difference between something that's generic and something that fits one device.12:08
rinigus... when you compare that to android?12:08
ThaodanI don't use official devices much honestly, I switched from XZ3 to Xperia 1. I would say the issues are not very device specific per se.12:09
ThaodanFor my usage my devices are fine the only problem I have is seeing which app consumes what, account sync being stuck or apps keeping the device awake for to long.12:11
rinigusThaodan: re app (or process)-specific consumption is missing, indeed. personally, I wouldn't set it as a priority, though :)12:43
piggzrinigus: you take the best jobs ... if you were busy, i was thinking of tackling the chum ui :)12:55
riniguspiggz: please do! I am fighting with maplibre a bit right now, so would be great if you look into it12:56
piggzrinigus: i was wondering, if we used json format, it would load directly into qml?12:56
riniguspiggz: json is pain to write, pain to read by human. let me point you to the code that needs to be changed... just a sec12:57
piggzits only a few extra "  " compared to yaml :D12:59
piggzand ,12:59
rinigusits parsing of https://github.com/sailfishos-chum/sailfishos-chum-gui/blob/main/src/chumpackage.cpp#L2613:00
riniguspiggz: ^13:00
riniguspiggz: and we have libs that will parse yaml for you. as in https://github.com/jbeder/yaml-cpp or maybe something even better13:01
piggzok, will try to look later, xmas decorations thas afternoon :/13:01
rinigusha! what kind of priority is that... xmas decorations are temp, chum is forever13:02
piggzrinigus: i will let you explain that to the family :D13:07
riniguspiggz: ... can't hear you over here ... too .. far :)13:07
riniguspiggz: ended up dealing with xmas decorations over here as well. irony16:13
piggzrinigus: i ended up also fitting a radiator, which wasnt a fun job16:15
rinigus:)16:16
ThaodanWould be nice to get some feedback or translations on communi-sailfish before making a new release for harbour18:32
Thaodanhttps://spdx.org/licenses/18:32
Thaodanhttps://github.com/communi/communi-sailfish/issues/15218:32
attahNice! Used it a couple of times with a separate aarch64 build, and it does what it should.18:39
Thaodanattah: I'll provide that aarch64 build in the next harbour release and a nightly/dev build18:43
attah<318:43
attahSorry i can't muster the will to contribute a sv translation... i don't even translate my own apps - maybe i should just switch language altogether18:45
Thaodanattah: np enabling translations is the easy part :D18:51
Thaodanrinigus: What's the policy on unstable/nightly releases in chum?19:21
ThaodanI want to a nightly build of communi-sailfish19:21
rinigusThaodan: we don't have any policy as far as I know. we prefer if developers are handling their own projects at :chum:testing . as for :chum proper, that goes via submit request from chum:testing to :chum. so, night builds in chum:testing would be fine19:23
rinigusmy development device is configured for chum:testing actually. that way I can check packages there as well19:24
Thaodanrinigus: That was the intend. About normal chum:  I think that could work too, shouldn't really be an entra work compared to harbour19:24
rinigusThaodan: regular chum is considered as stable and is for releases. so, it is for something that you would release in the store for end users19:25
ThaodanI want to push python-cairosvg to chum also to convert my graphics on the fly19:25
ThaodanThat would come at least before stable chum19:26
rinigusexcellent! looking forward19:26
Thaodanrinigus: I'll open a bug for repositories I need in main then19:28
Thaodan+19:28
rinigusThaodan: just one more note - we have few requirements that may make it touch more difficult. webhooks are not allowed and submission should have revision specified. that is all to ensure that there are no builds "behind the back" and they are reproducible.19:30
rinigusre repos requested: yes, please do.19:30
rinigussee checklist at https://github.com/sailfishos-chum/main/blob/main/Maintainer.md19:30
Thaodanrinigus: Sure, altrough webhooks should hide nothing behind the back. I'll package it normally in the tar_git anyway19:32
rinigusThaodan: I have to go now, sorry. will be afk for some time19:32
Thaodanrinigus: np19:36
Thaodanrinigus: https://github.com/sailfishos-chum/main/issues/3119:36
* piggz[m] uploaded an image: (748KiB) < https://matrix.org/_matrix/media/r0/download/matrix.org/kDNanZkuzNCbdvKiuOEktSUI/sailtrix-file.png >22:57
piggz[m]rinigus^^22:57

Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!