Previous day | Jump to hour: 01 02 03 04 05 06 07 08 09 10 11 12 13 14 15 16 17 18 19 20 21 22 23 | Next day

Seconds: Show Hide | Joins: Show Hide | View raw
Font: Serif Sans-Serif Monospace | Size: Small Medium Large

Click in the nick column to highlight everything a person has said.
The Logo icon identifies that the person is a core developer (has commit access).

#rockbox log for 2025-03-05

00:13:05***Saving seen data "./dancer.seen"
00:35:47 Quit Galois (Remote host closed the connection)
00:38:23 Quit thanosengine (Quit: WeeChat 4.5.1)
00:41:33 Join thanosengine [0] (~thanos@user/thanosengine)
01:00
01:16:30 Join Galois [0] (djao@efnet.math.uwaterloo.ca)
02:00
02:13:08***Saving seen data "./dancer.seen"
02:20:12 Quit othello7 (Quit: othello7)
02:53:31 Join amachronic [0] (~amachroni@user/amachronic)
03:00
03:28:44 Quit amachronic (Remote host closed the connection)
04:00
04:04:34 Quit jacobk (Ping timeout: 260 seconds)
04:05:46 Join jacobk [0] (~quassel@47-186-65-73.dlls.tx.frontiernet.net)
04:13:12***Saving seen data "./dancer.seen"
04:21:50 Join jjs0077018310196 [0] (~jjs007@host86-134-20-6.range86-134.btcentralplus.com)
05:00
05:03:10 Join amachronic [0] (~amachroni@user/amachronic)
06:00
06:13:14***No seen item changed, no save performed.
06:14:26 Quit Tonux (Quit: Bye)
06:16:03 Join Tonux [0] (~Tonux@193.32.127.200)
06:25:05 Quit braewoods (Ping timeout: 248 seconds)
08:00
08:13:18***Saving seen data "./dancer.seen"
08:14:13_bilgus_ g#6322
08:14:16rb-bluebotGerrit review #6322 at https://gerrit.rockbox.org/r/c/rockbox/+/6322 : FS #13538 export and import battery level tables WIP by William Wilgus
08:16:15_bilgus_so I have this in a decent place now, I decided to put the export of defaults into battery_bench, when run if 'battery_levels.default' doesn't exist and the default hasn't changed its created then.
08:17:04_bilgus_I'll have to add something about it in the manual
08:18:01_bilgus_later if someone wants to it could be all automated I'm probably not doing so any time soon..
08:19:34_bilgus_I think I'd like to get rid of the multiple battery types for the what 2 targets that have the option?
08:21:42_bilgus_I couldn't figure out how to share the device_battery_tables struct with plugins without funky ifdefs unfortunately
08:24:21 Join thanosen1 [0] (~thanos@user/thanosengine)
08:25:54 Quit thanosen1 (Client Quit)
10:00
10:00:09speachyI think there's just a single target with multiple types now.
10:01:30speachy(ie the xduoox3) and it's highly improbably anyone actively using one of the early production run still has the original battery.
10:13:21***Saving seen data "./dancer.seen"
10:28:03 Quit amachronic (Ping timeout: 276 seconds)
10:29:29 Join amachronic [0] (~amachroni@user/amachronic)
11:00
11:09:45 Join braewoods [0] (~braewoods@user/braewoods)
11:20:01 Quit jacobk (Ping timeout: 248 seconds)
11:51:14 Quit braewoods (Read error: Connection reset by peer)
11:52:27 Join braewoods [0] (~braewoods@user/braewoods)
11:56:59 Quit amachronic (Quit: amachronic)
12:00
12:11:17 Join jacobk [0] (~quassel@utdpat241060.utdallas.edu)
12:13:23***Saving seen data "./dancer.seen"
12:28:48 Join OlsroFR [0] (~OlsroFR@user/OlsroFR)
12:29:47OlsroFRHey all and hey Christian. https://gerrit.rockbox.org/r/c/rockbox/+/6410 I just pushed a new patch: this is my today work. I noticed that with multiselection enabled, moving entire CDs was kinda complicated if you don't really know what is the first title & the last title of the track from each of your CDs...
12:30:12OlsroFRThis option is not dependent of the multiselection code to be merged, though, its full interest in terms of practical use-cases are linked to the multiselection
12:30:45OlsroFRI recommend also disabling the "Show indexes" option in the playlists viewer, so you don't see touch much numbers at the beginning of the tracks
12:30:57OlsroFR(when this new option is active)
12:41:26 Quit OlsroFR (Quit: Client closed)
12:58:12 Quit jacobk (Ping timeout: 276 seconds)
13:00
13:36:32 Join othello7 [0] (~Thunderbi@pool-100-36-176-164.washdc.fios.verizon.net)
13:54:58 Join chris_s [0] (~chris_s@2a09:bac2:2b09:2496::3a5:f)
14:00
14:00:16chris_slooks like the sysfont size change breaks some themes, according to iPodVT on the forums
14:07:17speachyIn what way, the themes are expecting to use sysfont in places?
14:07:58chris_sApparently the theme expects the statusbar to be no higher than 8px and so parts of it overlap with the custom UI
14:08:18chris_sWhich must have been a problem on the Eros Q already
14:08:23speachy...exactly
14:09:14speachyI wonder if there's a decent way to detect this in checkwps
14:11:47speachyI'm of two minds here, on one hand "it's okay to break themes that aren't written properly" but on the other, "this doesn't look good"
14:13:06chris_sI would be surpirsed if there was an easy way to check for this. In that case we might want to detect overlapping viewports in general, which tend to cause issues. Plus, the themes of a certain screen size appear for different devices, which can have different sysfonts (as was the case here, apparently)?
14:13:24***Saving seen data "./dancer.seen"
14:13:51chris_sthere's a similar issue when different devices with the same screen size offer different settings which the theme uses is some way
14:13:58speachydifferent screen _resolution+bitdepths_ to be pedantic; that's not the same as "size"
14:14:43chris_sright
14:16:10speachyIt seems to me that the "correct" way to fix this in a theme is to explicitly enumerate every font used, and make no assumptions about the sysfont size.
14:17:22speachyie you shouldn't be using sysfont for any text you're displaying anyway, as it's essentially broken for non-latin1 charsets.
14:17:42chris_sis that possible for the status bar (if you're using the built-in one)? honestly not sure
14:17:46speachy(might only be limited to ASCII, not sure)
14:18:11speachythe default status bar is sized at SYSFONT height.
14:18:59chris_syou can't use that in a theme though, can you?
14:19:16speachythemes don't have to override the status bar
14:19:40speachyso effectively it means the main viewport for the content can vary in height depending on the particular sysfont used.
14:19:46speachyUgh
14:21:11chris_syeah
14:21:28speachyeven if we revert this or decouple the sysfont from the statusbar height (which may be easier said than done) themes still shouldn't be making assumptions
14:27:00chris_sI guess they have to assume *some* (maximum) height for the status bar. It didn't occur to me at the time, when testing the patch, that themes would even choose to display the default status bar, instead of rolling their own, but of course some do.
14:28:12chris_sThe increased font size seems mostly useful to me for the keyboard, specifically
14:28:16speachyUgh.
14:30:01speachyat this point the prudent step is to revert (including erosq) and reassess how to move forward
14:30:28speachyyeah there are a ton of themes that don't touch the sbs
14:36:51chris_sI guess. Although *most* 320x240themes for the iPod seem to draw their own status bar, I'm actually trying to find one that even demonstrates the issue atm
14:37:57speachyhmm, cabbiev2's sbs is 8px hardcoded.
14:41:48chris_scabbie looks fine to me, apart from the icons being too small / not being centered? ok here is one that expects 8px for example: https://themes.rockbox.org/index.php?themeid=2311&target=ipodvideo
14:42:41speachyI mean cabbie is effectively what's used when the theme has no sbs
14:43:40chris_sI think the issue is when there is an SBS but you have %wi in a viewport that's too small
14:43:54chris_shttps://download.rockbox.org/daily/manual/rockbox-ipodvideo/rockbox-buildap4.html#x22-462000D.1
14:44:38chris_sor viewports overlapping
14:45:33chris_sbut in the case of that metal theme, even the background picture relies on the status bar being a certain height:D
14:55:28speachyI just pushed a patch that reverts back to the 8px sysfont on those targets. It can go back in once we figure out a better path forward.
14:55:38chris_sok
14:57:26speachyeven if we just put it back in as-is (post 4.0?) I would really like to have checkwps complain.
15:00
15:06:05chris_ssomeone may want to verify that they keyboard font is still somewhat readable on the eros q due to its smaller screen size than the iPods
15:06:29speachythat was the original reason I upped the font size there (or at least I think it was me..)
15:23:20 Quit IPG (Ping timeout: 265 seconds)
15:32:31 Join jacobk [0] (~quassel@utdpat241106.utdallas.edu)
16:00
16:03:49 Quit jacobk (Ping timeout: 260 seconds)
16:13:25***Saving seen data "./dancer.seen"
16:56:35 Join PheralSparky [0] (~S|h|a|w|n@user/shawn/x-4432647)
18:00
18:13:29***No seen item changed, no save performed.
18:20:02 Quit hactar|ant (Read error: Connection reset by peer)
18:21:24 Join hactar|ant [0] (~zem@97-115-105-15.ptld.qwest.net)
18:37:26 Quit chris_s (Quit: Client closed)
18:50:10 Quit hactar|ant (Ping timeout: 252 seconds)
18:54:39 Join hactar|ant [0] (~zem@97-115-105-15.ptld.qwest.net)
18:55:49 Join drew` [0] (~drew@user/drew)
18:57:02 Quit drew (Ping timeout: 244 seconds)
18:57:06 Nick drew` is now known as drew (~drew@user/drew)
19:00
19:46:05blebspeachy: a few days ago you were talking about how the ESP32 (used by the tangara) depends on a lot of proprietary firmware
19:46:27blebare there alternatives that don't require proprietary firmware?
19:48:59 Join massiveH [0] (~massiveH@2600:4040:a982:5400:8022:640e:3060:16a6)
19:57:21 Join IPG [0] (~InvoxiPla@2a06:5902:19c0:9900:98e9:71ff:45f7:4e76)
20:00
20:13:30***Saving seen data "./dancer.seen"
20:43:36 Quit PheralSparky (Read error: Connection reset by peer)
20:44:15 Quit IPG (Read error: Connection reset by peer)
22:00
22:13:33***Saving seen data "./dancer.seen"
22:17:08 Join jacobk [0] (~quassel@47-186-65-73.dlls.tx.frontiernet.net)
22:18:33_bilgus_bleb, I imagine most of the targets running native
22:59:59blebnative?
23:00
23:02:41bleboh u mean daps that rockbox supports?
23:03:23blebI was more asking about microcontrollers with wifi that u could build an open source dap on top of
23:07:54 Quit massiveH (Quit: Leaving)

Previous day | Next day