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 2024-08-29

00:37:36 Join dconrad [0] (~dconrad@152.117.104.217)
00:42:48 Quit dconrad (Ping timeout: 276 seconds)
01:00
01:06:54 Join dconrad [0] (~dconrad@152.117.104.217)
01:23:53***Saving seen data "./dancer.seen"
02:00
02:18:46 Quit dconrad (Remote host closed the connection)
02:31:37 Join dconrad [0] (~dconrad@152.117.104.217)
02:36:28 Quit dconrad (Ping timeout: 265 seconds)
02:51:27 Quit Jinx (Ping timeout: 265 seconds)
03:00
03:23:55***Saving seen data "./dancer.seen"
03:36:09 Join halloy6399 [0] (~halloy639@aaubervilliers-198-1-21-253.w92-159.abo.wanadoo.fr)
03:36:31 Nick halloy6399 is now known as OlsroFR (~halloy639@aaubervilliers-198-1-21-253.w92-159.abo.wanadoo.fr)
03:37:00OlsroFRI still cannot upload my theme, getting same error on the site: " Disable main menu scrolling is not a allowed theme setting."
03:37:45 Quit Skyrider (Quit: ZNC - 1.6.0 - http://znc.in)
03:43:52 Join Skyrider [0] (~Skyrider@static.219.32.201.138.clients.your-server.de)
03:47:24 Quit OlsroFR (Remote host closed the connection)
04:00
04:45:04 Join dconrad [0] (~dconrad@152.117.104.217)
04:49:25 Quit dconrad (Ping timeout: 248 seconds)
05:00
05:04:49 Quit jacobk (Ping timeout: 260 seconds)
05:05:21 Join jacobk [0] (~quassel@47-186-105-237.dlls.tx.frontiernet.net)
05:23:56***Saving seen data "./dancer.seen"
05:30:07 Join reb [0] (~reb@80.208.34.77)
05:42:25rebhey, i don't normally do any sort of collaborative work like this with others, so my apologies for being a bit behind (a noob). to show graditude for rockbox re-igniting my love of DAPs and overall quality of life improvements of my device, i decided to, at the very least, contribute in translation efforts. i saw the lithuanian language has been
05:42:26rebforgotten since 2009, haha. so, as a native from LT, i've translated some chunks. but i am not entirely sure what's some appropriate places to submit the work i've done (excluding sending it as a bugfix...i am ESL so i was confused by the language barrier if that'd be the way to go. aka i cannot decide if "create patch" or "create full" is the way
05:42:26rebto go). let me know if i should do this anywhere elsewhere! also, i plan to try and translate 90% (or more) for LT in the future, especially if no other native speakers are available to help out in this area :) so far i just added a lot of missing words, and corrected a couple of outdated LT terminology/incorrect use of words in the context of
05:42:27DBUGEnqueued KICK reb
05:42:27rebgrammar.
05:44:05rebagain, my apologies for struggling to comprehend english directions, but i am trying my best as an ESL speaker ^^;
06:00
06:49:12 Quit chamlis (Remote host closed the connection)
06:49:26 Join chamlis [0] (~chamlis@user/chamlis)
06:58:31 Quit chamlis (Remote host closed the connection)
06:58:46 Join chamlis [0] (~chamlis@user/chamlis)
07:00
07:02:12 Quit chamlis (Remote host closed the connection)
07:02:27 Join chamlis [0] (~chamlis@user/chamlis)
07:03:29 Quit chamlis (Remote host closed the connection)
07:03:44 Join chamlis [0] (~chamlis@user/chamlis)
07:20:27 Join dconrad [0] (~dconrad@152.117.104.217)
07:23:59***Saving seen data "./dancer.seen"
07:24:15 Quit Moriar (Ping timeout: 246 seconds)
07:24:49 Quit dconrad (Ping timeout: 252 seconds)
07:29:30 Quit olspookishmagus (Ping timeout: 246 seconds)
07:31:11 Quit chamlis (Remote host closed the connection)
07:31:25 Join chamlis [0] (~chamlis@user/chamlis)
07:32:22 Quit chamlis (Remote host closed the connection)
07:32:36 Join chamlis [0] (~chamlis@user/chamlis)
07:37:35 Join halloy6399 [0] (~halloy639@aaubervilliers-198-1-21-253.w92-159.abo.wanadoo.fr)
07:37:49 Nick halloy6399 is now known as OlsroFR (~halloy639@aaubervilliers-198-1-21-253.w92-159.abo.wanadoo.fr)
07:38:18 Quit chamlis (Remote host closed the connection)
07:38:31 Join chamlis [0] (~chamlis@user/chamlis)
07:41:22OlsroFR_bilgus: https://gerrit.rockbox.org/r/c/rockbox/+/5903 https://gerrit.rockbox.org/r/c/rockbox/+/5901 I have processed all your review comments.
07:43:30 Quit chamlis (Remote host closed the connection)
07:43:45 Join chamlis [0] (~chamlis@user/chamlis)
07:44:57 Quit chamlis (Remote host closed the connection)
07:45:11 Join chamlis [0] (~chamlis@user/chamlis)
07:49:34OlsroFRand also added manual entries to explain that shuffling in the context of the database browser is going to pick random from the whole view if it exceeds the limit of the system
07:51:11edhelas_OlsroFR thanks for your awesome contributions !
08:00
08:00:07rb-bluebotBuild Server message: New build round started. Revision 054ba76d81, 337 builds, 9 clients.
08:00:07rb-bluebotMorse code cheat sheet, better use [of] pixels available on screen by William Wilgus
08:01:36_bilgusOlsroFR, morse keybpard is pushed I merged it with 5904 which can now be abandoned
08:01:39 Join olspookishmagus [0] (~pookie@snf-137798.vm.okeanos.grnet.gr)
08:03:43OlsroFRthank you ! 5904 is now abandoned
08:04:28_bilgusyou are still tring to do a string compare for that string in tagnav
08:04:58_bilgusi'm not doing a string compare in my directoy loop for a single word in a single file
08:05:19_bilguswell two words in a single file
08:06:12_bilgussee 2167 for an example of how to parse the tagnav file
08:06:36 Quit olspookishmagus (Ping timeout: 246 seconds)
08:07:58OlsroFROK, will look into it
08:08:07_bilgusthanks.
08:12:46_bilgusin tagtree.c (also in 2167) you see here: https://github.com/Rockbox/rockbox/blob/master/apps/tagtree.c#L1465 thats where you will check your tag but it won't be a strcmp but a enum match to https://github.com/Rockbox/rockbox/blob/master/apps/tagtree.c#L362
08:12:48rb-bluebotBuild Server message: Build round completed after 761 seconds.
08:12:50rb-bluebotBuild Server message: Revision 054ba76d81 result: All green
08:12:59rb-bluebotBuild Server message: New build round started. Revision edd607c392, 337 builds, 9 clients.
08:12:59rb-bluebotFS #13473: Update Korean translation (Hoseok Seo) by Solomon Peachy
08:15:04 Nick jn_ is now known as jn (~quassel@user/jn/x-3390946)
08:17:44 Join olspookishmagus [0] (~pookie@83.212.124.75)
08:25:18rb-bluebotBuild Server message: Build round completed after 740 seconds.
08:25:20rb-bluebotBuild Server message: Revision edd607c392 result: All green
08:38:24 Join thanosengine [0] (~thanos@user/thanosengine)
09:00
09:00:30 Quit thanosengine (Ping timeout: 246 seconds)
09:15:46 Quit rogeliodh91 (Quit: The Lounge - https://thelounge.chat)
09:16:08 Join rogeliodh91 [0] (~rogeliodh@rogeliodh.dev)
09:24:01***Saving seen data "./dancer.seen"
09:47:50 Quit reb (Quit: Connection closed)
09:47:53speachyreb: Generally we prefer the patch download but given how outdated that specific translation is, the full one is better.
09:48:11speachythe full one is also useful if you want to work on it offline
09:52:25 Join reb [0] (~reb@80.208.34.77)
09:56:43 Quit reb (Client Quit)
10:00
10:56:24 Quit TorC (Ping timeout: 276 seconds)
10:59:26 Join TorC [0] (~Tor@fsf/member/TorC)
11:00
11:14:28 Join jj5_ [0] (~jj5@100.80.216.139.dynamic.dsl.dv.iprimus.net.au)
11:14:58 Quit jj5 (Ping timeout: 248 seconds)
11:15:02 Nick jj5_ is now known as jj5 (~jj5@100.80.216.139.dynamic.dsl.dv.iprimus.net.au)
11:23:35 Quit othello7 (Quit: othello7)
11:24:03***Saving seen data "./dancer.seen"
11:26:23 Join othello7 [0] (~Thunderbi@pool-100-36-176-164.washdc.fios.verizon.net)
11:32:20 Join thanosengine [0] (~thanos@user/thanosengine)
11:48:02 Quit ParkerR (Ping timeout: 248 seconds)
11:56:58 Join ParkerR [0] (znc@znc.withg.org)
12:00
12:00:52 Join Jinx [0] (~Jinx@user/jinx)
12:06:27 Quit paulk (Ping timeout: 244 seconds)
12:14:24 Join paulk [0] (~paulk@about/aquilenet/user/paulk)
12:15:56 Quit thanosengine (Quit: WeeChat 4.3.1)
12:16:34 Quit chamlis (Remote host closed the connection)
12:17:02 Join chamlis [0] (~chamlis@user/chamlis)
12:36:30OlsroFR_bilgus: https://gerrit.rockbox.org/r/c/rockbox/+/5911 updated, ready to review. I changed the whole parsing logic with some more refactoring. I also tested on hardware to check for regressions, and all seems fine
12:47:42 Quit OlsroFR (Ping timeout: 272 seconds)
13:00
13:06:35 Join davisr [0] (~davisr@fsf/emeritus/davisr)
13:24:06***Saving seen data "./dancer.seen"
14:00
14:04:38sporki think if you write g#5911 the bot will respond
14:04:41rb-bluebotGerrit review #5911 at https://gerrit.rockbox.org/r/c/rockbox/+/5911 : Reworks to the shuffle system to improve performance and allow fast shuffling from a big library (but this work for all database views) by Paul Sauro
14:14:51 Join halloy6399 [0] (~halloy639@aaubervilliers-198-1-21-253.w92-159.abo.wanadoo.fr)
14:19:13 Quit halloy6399 (Changing host)
14:19:13 Join halloy6399 [0] (~halloy639@user/OlsroFR)
14:19:34 Quit halloy6399 (Remote host closed the connection)
14:20:04 Join halloy6399 [0] (~halloy639@aaubervilliers-198-1-21-253.w92-159.abo.wanadoo.fr)
14:20:32 Quit halloy6399 (Remote host closed the connection)
14:21:34 Quit davisr (Quit: yeehaw)
14:23:05 Join OlsroFR [0] (~OlsroFR@user/OlsroFR)
14:25:23 Join OlsroFR81 [0] (~OlsroFR@aaubervilliers-198-1-21-253.w92-159.abo.wanadoo.fr)
14:25:28 Part OlsroFR81
14:25:59 Quit OlsroFR (Client Quit)
14:26:28 Join OlsroFR [0] (~OlsroFR@user/OlsroFR)
14:32:05OlsroFRJust tested to upload again my theme, still failing with error : "Disable main menu scrolling is not a allowed theme setting."
14:35:41speachywhat happens when you run checkwps on the theme manually?
14:37:20speachyah, I see, that's from a whitelist on the themesite itself..
14:38:15OlsroFRYeah. I just took a look on the tool "checkwps", he is not even able to check a .cfg file. Options are set only in the .cfg file, when you apply the theme
14:45:07speachyshould work now
14:45:31OlsroFRmy body is ready
14:45:32speachyugh, this really shouldn't be part of the theme site.
14:47:34OlsroFRmmm I just upload my first theme but the result was a blank page
14:47:38OlsroFRis that intended ?
14:48:22OlsroFRI just uploaded Minimless for the iPods 1G to 4G
14:49:51_bilgusOlsroFR, that patch set looks much better 3 little things in it but I still don't like that 256k static alloc It'll be a few days before I can sit down and check it out but I'll look into doing an allocation from ram and see if that will work better
14:50:54OlsroFROkay. I am going to look into your feedbacks and maybe try to understand by myself how to do it dynamically using RAM allocation
14:53:10_bilgusI wouldn't be against it so much if it wasn't fallow the rest of the time but as is thats like 15-20 seconds of playback on a mp3 permanently removed
14:53:45_bilguscore_alloc is the function for dynamic allocs
14:53:55_bilguswe don't use malloc
14:54:42edhelas_What is the minimum amount of RAM that Rockbox can handle ? Is there some devices that have something like 32mb or less ?
14:55:15_bilgus2 on the original clip
14:55:29_bilgus8 on the clip+ zip fuzev1 v2
14:55:41_bilgusI think the fuze+ is 32
14:55:59edhelas_Damn :D So those 256k are actually quite a lot
14:56:04_bilgusyes
14:58:16_bilgusI can't speak for everyone here but I really do try to think about others experiences when we add stuff like what percentage of users actually care about this versus the overhead everyone will get from it
14:59:01_bilguslike if its a global benefit then yes that makes sense
14:59:25_bilgusI like the patch set idea just hope we don't eat so much ram for it
14:59:49OlsroFRWe can reduce that buffer the something lower like 64K. The code is protected against buffer overflow. But it will not go over 64000 tracks. But how many people in the community have more than 64K tracks ? ;)
15:00
15:00:43OlsroFRMy whole music library is "just" around 25000 tracks currently
15:01:30_bilguswell if you look at the playlist code there is a max there too and I think I have somewhere north of 60000 in my db
15:02:20OlsroFRhttps://forums.rockbox.org/index.php/topic,52337.msg242319.html#msg242319 a person here claimed to have 65000 songs on his ipod classic lol
15:02:56edhelas_Once I'll put the correct SD card I'll reach that, got something like 45000 songs
15:03:07edhelas_Wondering how much time it will take to build the DB 😬
15:03:43OlsroFRon my ipod mini it takes around 1 hour to build the database on device. But once it is built, it just work... until I want to add more songs ;)
15:04:02OlsroFRif you have the ipod classic 6th to 7th gen, it will be of course much faster because the CPU is much faster
15:05:34edhelas_Would it maybe be interesting to have a tool to build it on the PC ?
15:05:45speachyit exists, though you have to compile it yourself.
15:05:51speachy('dbtool')
15:06:09OlsroFRah interesting. I was using the simulator to do that and it was just very slow
15:06:11edhelas_Oh !
15:06:34sporki thought i read the tool/sim did not work building the db 'because of endianess'
15:06:52sporkon some forum post, so might be very wrong
15:06:58_bilgusthe db is endian agnostic
15:07:13sporkthat would make that argument nonsense then
15:07:22_bilgusdepends
15:07:57_bilgussomething might be different between them still or mybe thats an old post
15:08:32_bilgusbut unless your folder layout is the same..
15:09:54speachyI don't know if are necessarily any differences in the on-disk db format between individual players, but the dbtool has _always_ been a per-target build instead of a generic tool.
15:10:07speachyso take that for what it's worth.
15:10:18_bilgusanyway OlsroFR if you figure something out before I get to it tahts fine or if we just can't figure a way around it maybe we can dial it back for the lower rm targets
15:10:41_bilguslower ram*
15:12:18_bilgusdepending on what contexts it'll be called in we might be able to steal the plylist buffer
15:12:31_bilgussorry PLUGIN buffer
15:12:49OlsroFRWould be a very good compromise. I will need to take some time to do some research about this.
15:13:05OlsroFRBut well let's be very carefull about stealing another buffer, this might create very weird bugs depending of the library size
15:14:34_bilgusthat pretty much depends on if its exported to plugins and it very well may be and in that case we could require a buffer supplied by caller
15:15:21_bilgusthe other option would be to fill a smaller buffer and when full randomly add anything that doesn't match
15:17:05_bilgusbasically the buffer is full so if this new file isn't already here we do a rand and if it matches we add it anyway
15:18:10OlsroFRMmm not sure to perfectly visualize your solution, but we need to be carefull to not add 2 times the same song. Right now the code is clever enough to never do this and pick really random songs all along the library by avoiding duplicates
15:18:15 Quit Galois (Remote host closed the connection)
15:18:54_bilguswelland thats the downside of that solution if you run out of room you might get a double song
15:21:16_bilgusI wonder since our rand function is deterministic in regards to the seed you could actually throw away part of the field
15:22:38_bilgusyou would just keep the number of iterations and you could trade time for space once you ran out
15:24:09***Saving seen data "./dancer.seen"
15:26:08OlsroFRI will test different implementations and will test the impacts on hardware
15:26:26_bilgusbasically you know the seed and now that the buffer is full you throw away the whole thing and fill it again once thats done you then cull entries by checking the output of rand with the original seed and to the numer of iterations
15:28:17_bilgusso lets say you had 100 entries they are all unique and added you throw aya the unique buffer and fill it with the next 100 entries that are all unique (to each other) and now you check against rand (run 100 times)
15:28:42_bilgusany that match you strike from the buffer
15:29:02_bilgusnow the next 100 gets filled and check against rand(200x)
15:29:58_bilgusit will get exponentially slower so needs to be limited or at least cancel-able
16:00
16:04:36OlsroFRDynamic allocation seems to work. I just found the constant AUDIO_BUFFER_RESERVE which seems to be the currrent reserve to store non-audio dynamic data
16:04:59OlsroFRwhich is, 256k, as you said, it can represents 20s of mp3 file
16:06:19OlsroFRwhat can be frustrating with it is that I have no control on it. If any other part of the code has taken some memory from it, allocation may fail completely
16:08:30 Join Galois [0] (djao@efnet.math.uwaterloo.ca)
16:08:45OlsroFRthis memory space is currently shared with the tagtree
16:09:24OlsroFRSo I do not like the idea to take memory here
16:13:57edhelas_Is it wise to have the same buffer on devices that have flash drive than HDD ? I mean the buffer and IO access is way faster with flash storage no ?
16:33:22 Quit jacobk (Ping timeout: 248 seconds)
16:38:46_bilgustypically the devices with HDD have gobs of ram
16:49:26_bilgus OlsroFR as far as I can see the only tagtree function exported to plugins is tagtree_subentries_do_action() by the properties plugin
16:51:22_bilgusfrom what I can see it doesn't touch insert_all_playlist() so in theory you would be fine to take whatever is left of the plugin buffer (which could still be 0 we have TSR plugins that might choose not to give it up)
16:51:28OlsroFR_bilgus : initialize_tagtree is taking up space in the core buffer. Probably all time if you decide to keep it in RAM ? (which is the most optimized choice)
16:53:25_bilgusit shrinks it when its done
16:54:13_bilgusnot sure why that matters though?
17:00
17:02:32_bilgusso i'm looking here https://gerrit.rockbox.org/r/c/rockbox/+/5911/3/apps/tagtree.c#2173
17:03:38_bilgusnot the codestyle is still different than the rest of the code but besides the point you have a check for playlist == NULL to do your randomizing
17:04:09_bilgusbut you are reading from the current playlist and later writing them to the same playlist?
17:04:26OlsroFRI noticed that playlist is always NULL in the context of inserting from the tag database
17:06:31_bilguswell yeah because its the current playlist later https://github.com/Rockbox/rockbox/blob/master/apps/playlist.c#L2625
17:09:13_bilgusno nevermind you are just pulling the maxsize from it not the actual indices
17:14:22_bilgusOlsroFR, I think with the way this is set up you could actually just look if max_available_space is too small and randomly skip indices
17:15:32OlsroFR_bilgus this way it will not be truly random; I have thought about this. The issue with true random skipping is that the playlist will grow until the limit and stop growing when the limit will be reachen.
17:15:55OlsroFRBecause of this behavior, by doing this way, this "randomness" will tend to "random" only the first tracks
17:16:38OlsroFRExemple : If you have a system limit of 2000 and you put 50% of chances to skip the current index, it means that statically you will get true randomness only between the first 4000 songs of your library
17:18:16_bilgustag_cache_retrieve will return whatever indice you choose
17:19:13OlsroFRAh I get what you mean. That was my first try on developing this. From what I can remember, for a reason that I don't know, the performance was very very slow
17:19:16_bilgusif you know that there is not enough room ahead of time if you choose to skip every 0-6 songs you effectively have a random playlist still and a larger sampling of the whole library
17:19:20OlsroFRAlso, another issue is that you may get duplicates
17:19:49_bilgusyou will get duplicates anyway thats why it has a uniqbuf
17:20:21OlsroFRright now with the current system there is no possible duplicates, it's checking for it, look in the while (true) loop
17:20:41OlsroFR(with what I developed and pushed for review)
17:20:50_bilgussure but you have reinvented the uniq buf with a fallow buffer
17:21:38OlsroFRI wanted something perfect just like the Stock OS can do about this feature. When you mix your library on Stock OS, you never get duplicates
17:22:20_bilgusyou don't get duplicates currently either not disputing that
17:23:09_bilgusretrieve_entries already does this
17:24:11***Saving seen data "./dancer.seen"
17:24:50_bilgusso all you need to do is make sure theres enough uniqbuf and skip a random amount between 0 and n and you need to insure n isn't so large that you don't have enough entries left
17:26:07OlsroFRnoted
17:26:53_bilgusnot saying i'm not wrong but i'm pretty sure this will do what you want with minimal extra overhead (might increase uniqbuf)
17:27:38_bilgusbut thats fine atleast it has 2 users
17:30:08OlsroFRI am still currently coding something, I feel like I am following the good path. I will push it for review when it will be done
17:30:12OlsroFRprobably in one hour or two
17:31:49_bilgusI'm headed out but i'll check back later this eve I check gerrit so just make sure you use −−amend and I'll see it
17:38:44 Join Moriar [0] (~moriar@107-200-193-159.lightspeed.stlsmo.sbcglobal.net)
19:00
19:08:18 Join dconrad [0] (~dconrad@152.117.104.217)
19:24:15***No seen item changed, no save performed.
19:56:03 Join massiveH [0] (~massiveH@2600:4040:a982:dc00:38d6:2ca9:2d4f:f66b)
21:00
21:24:19***No seen item changed, no save performed.
22:00
22:05:25dconrad_bilgus I like the idea of the HAVE_BOOTDATA option, I feel like this could be useful to other ports in the future without much work
22:05:40dconradIt appears to work as promised
22:11:45 Quit Moriar (Ping timeout: 276 seconds)
22:15:36OlsroFR_bilgus https://gerrit.rockbox.org/r/dashboard/self new patches submitted. It was tought, I am now tired, but happy. Reduced the memory consumption to divide it by 256 basically ;)
22:17:18OlsroFRI didn't use the dynamic buffer, for so little space it just do not worth it anymore and might cause issues since there is yield() and the core buffer is not recommended to use in this case. I also fixed some bugs. I've did many hours of coding and testing. I am going to sleep now.
22:19:43OlsroFRWith this system of segments, the random is slightly less random especially for the last segment that is processed. It's the very little compromise of this patch. But it should be more than good enough for the use case
22:20:04 Quit OlsroFR (Quit: Connection closed)
22:39:49 Join OlsroFR [0] (~OlsroFR@user/OlsroFR)
22:40:05 Quit OlsroFR (Client Quit)
22:49:09 Join OlsroFR [0] (~OlsroFR@user/OlsroFR)
22:49:30OlsroFRI could finally successfully upload my 4 themes, thanks again for the help! I am so happy to share those
22:49:57OlsroFRhttps://themes.rockbox.org/index.php?target=ipodmini2g https://themes.rockbox.org/index.php?target=ipod4g
22:50:54 Quit OlsroFR (Client Quit)
23:00
23:22:11 Quit massiveH (Quit: Leaving)
23:24:21***Saving seen data "./dancer.seen"
23:43:15 Join jacobk [0] (~quassel@47-186-105-237.dlls.tx.frontiernet.net)

Previous day | Next day