Downloads
release
dev builds
extras
themes
Documentation
manual
wiki
device status
Support
forums
mailing lists
IRC
Development
bugs
patches
dev guide
translations
Search
Donate
1 Hour
1 Day
1 Week
1 Month
Forever
Login with username, password and session length
News:
Welcome to the Rockbox Technical Forums!
Rockbox Technical Forums
Support and General Use
User Interface and Voice
Make playlists use the database info of a song instead of its file name
« previous
next »
Print
Pages: [
1
]
2
Author
Topic: Make playlists use the database info of a song instead of its file name (Read 6769 times)
nezzled
Member
Posts: 13
Make playlists use the database info of a song instead of its file name
«
on:
July 01, 2024, 01:17:22 AM »
Hello! I would like to sync music with iTunes to use both Rockbox and the default iTunes firm, but because iTunes makes gibberish file names it would make navigating playlists annoying. Is there a way to make them use the embedded tags/database info instead of their file names? I know I can technically work around this by renaming them but I would prefer to do it the other way.
Logged
Bilgus
Developer
Member
Posts: 1008
Re: Make playlists use the database info of a song instead of its file name
«
Reply #1 on:
July 05, 2024, 04:05:04 PM »
the problem here is playlists are independent of the database system and looking up the tag info for each file would likely make it slow but I'll look into it
Logged
Bilgus
Developer
Member
Posts: 1008
Re: Make playlists use the database info of a song instead of its file name
«
Reply #2 on:
July 05, 2024, 05:16:33 PM »
Here is the first try at a patch that does this
ATM it reads every file from disk which is only when you are in the viewer OFC
https://gerrit.rockbox.org/r/c/rockbox/+/5807
if you let me know what device I can get you a build to try
Updated a bit:
this thing would be pretty slow with large playlists on a slow disk
in that case hopefully you are loading your database to RAM
and this will take advantage of it to speed up the access
«
Last Edit: July 09, 2024, 08:24:54 AM by Bilgus
»
Logged
Bilgus
Developer
Member
Posts: 1008
Re: Make playlists use the database info of a song instead of its file name
«
Reply #3 on:
July 09, 2024, 08:25:26 AM »
This is now in the latest dev version
«
Last Edit: July 13, 2024, 12:55:10 AM by Bilgus
»
Logged
Bilgus
Developer
Member
Posts: 1008
Re: Make playlists use the database info of a song instead of its file name
«
Reply #4 on:
July 13, 2024, 12:56:29 AM »
After optimizing this for a while and getting it working decently with on disk playlists its so terribly slow when trying to view the current playlist I might as well just remove it alltogether
Logged
Frankenpod
Member
Posts: 759
Re: Make playlists use the database info of a song instead of its file name
«
Reply #5 on:
July 13, 2024, 08:08:23 AM »
Seems to work acceptably for me (7th gen ipod, flash modded, database loaded to ram).
As most of my files are named according to the content anyway, it's not really a huge difference, but just tried it on some that are only named "track01" "track02" etc, and it seems to work and playlists appear in perfectly acceptable time. It's useful for such tracks that I ripped a bit too lazily. As it's a user-selectable option I'd vote for keeping it.
Possibly I haven't tried it on sufficiently long playlists though?
(I'm assuming this is the thing that comes under general settings/playlists/playlist viewer settings/"Track display")
«
Last Edit: July 13, 2024, 08:10:32 AM by Frankenpod
»
Logged
Bilgus
Developer
Member
Posts: 1008
Re: Make playlists use the database info of a song instead of its file name
«
Reply #6 on:
July 13, 2024, 09:17:15 AM »
the main issue I have with this is all the things I have to do to make it work (decently) for anyone else
for instance, making the number of tracks buffered [lower] makes the wait acceptable on first load but it quickly makes scrolling annoying
fix that by keeping track of how long we took to load and start tweaking how many we load per chunk
track search uses the same settings and now they do different things, time for setting flags or yet another setting
IDK I gave it a try I feel that this will cause more issues than its worth, I would only consider enabling the option if the playlist was already in ram
try looking at that playlist on first boot before its in ram bet you won't like the wait
I think I'd rather give you a way to add tags to playlist or something
«
Last Edit: July 13, 2024, 09:19:17 AM by Bilgus
»
Logged
Frankenpod
Member
Posts: 759
Re: Make playlists use the database info of a song instead of its file name
«
Reply #7 on:
July 13, 2024, 10:14:30 AM »
Fair enough, your decision. Maybe the performance depends on the target (available RAM?). Plus I haven't tried any particularly long playlists.
Could it be done so it can only be selected if the database is successfully cached in RAM?
Logged
Bilgus
Developer
Member
Posts: 1008
Re: Make playlists use the database info of a song instead of its file name
«
Reply #8 on:
July 13, 2024, 10:25:59 AM »
I'm testing with playlist containing 10000+ tracks and yes the available ram makes all the difference in the world
I really don't like adding a bunch of specialized code if I can help it its hard to keep up with
ATM I think the way forward is to support EXTM3u
chris_s found a FS patch that has most of what I want already
https://www.rockbox.org/tracker/task/7652
Give me a few days the current patch needs updated and I've already identified a few issues to work thru
after I get that working I'll work on a plugin to allow conversion between the two types
Logged
Frankenpod
Member
Posts: 759
Re: Make playlists use the database info of a song instead of its file name
«
Reply #9 on:
July 13, 2024, 10:32:39 AM »
To be honest, it's possible I've misunderstood this thing entirely.
Just tried it on a playlist of podcasts (only about 150 entries), and I don't actually include podcasts in the database (podcast directory has a 'database.ignore' file in it), yet it still seems to work.
The podcast file names are different from the titles (because I downsample them to lower bitrates and rename them to save space on the ipod) so it's definitely showing the title tag rather than the file name in the 'current playlist' display, if I select that option in the settings. As that's not in the database I don't actually understand how it's doing that.
Logged
Bilgus
Developer
Member
Posts: 1008
Re: Make playlists use the database info of a song instead of its file name
«
Reply #10 on:
July 13, 2024, 10:44:37 AM »
what the [removed] patch does is parses/reads the metadata from the file directly on disk if the db is not in-ram or if it doesn't find the track
what I'm likely going to do is allow extended m3u playlists so the data is in the playlist and then you only have to look at one (much smaller/denser) file
the other option is to make cue sheets and read the data from those (similar idea but separate files)
Logged
Bilgus
Developer
Member
Posts: 1008
Re: Make playlists use the database info of a song instead of its file name
«
Reply #11 on:
July 13, 2024, 10:55:55 AM »
so two options:
First: use exm3U files
advantages:
single file
only generated once
new playlist can be stored in the format
playlist searches become much more versatile
most of the code is already in FS
disadvantages:
not backwards compatible (plugin to convert between the two)
slower to parse
Second: use cue sheets (db of track info per playlist)
advantages:
doesn't touch the playlist (backwards compatible)
most of the code is already in RB
disadvantages:
clunky (you need to keep track of two files and they might diverge)
need to generate the cue sheet before using
Logged
Frankenpod
Member
Posts: 759
Re: Make playlists use the database info of a song instead of its file name
«
Reply #12 on:
July 13, 2024, 11:35:08 AM »
Well, I've no preference between those two.
When you say 'not backwards compatible', you mean in relation to any existing playlists people have already created on the player itself?
I mostly have playlists exported from media monkey, and they are all extended format anyway. Playlists created on the player are just temporary as far as I'm concerned, so I wouldn't, myself, be bothered about keeping compatibility in that regard. Maybe other users would, though.
Logged
Bilgus
Developer
Member
Posts: 1008
Re: Make playlists use the database info of a song instead of its file name
«
Reply #13 on:
July 13, 2024, 06:11:04 PM »
not backwards compatible as in if a playlist is made with the latest it woldn't work on older versions (3.15 for instance)
like I said thats not such a concern but still a disadvantage
are you able to use those exm3u playlist in rb directly as a regular playlist? if so then thats great and I think ill move forward with option 1
«
Last Edit: July 13, 2024, 06:26:49 PM by Bilgus
»
Logged
Frankenpod
Member
Posts: 759
Re: Make playlists use the database info of a song instead of its file name
«
Reply #14 on:
July 13, 2024, 07:06:54 PM »
Not at home, so can't check to see what exactly I set media monkey to export. But looking at the playlists I have on the player in a text editor, they start with #EXTM3U
and have entries like
#EXTINF:230, R.E.M. - Finest Worksong
..\Music\R.E.M\Document\01 Finest Worksong.mp3
So I assume those are exm3U playlists? And RB doesn't seem to have any problems with them.
Logged
Print
Pages: [
1
]
2
« previous
next »
Rockbox Technical Forums
Support and General Use
User Interface and Voice
Make playlists use the database info of a song instead of its file name
SMF 2.0.19
|
SMF © 2021
,
Simple Machines
Rockbox Privacy Policy
XHTML
RSS
WAP2
Page created in 0.054 seconds with 18 queries.