New Minion 2 Alpha: 1.99.23

0 views
Skip to first unread message

Chris S

unread,
Mar 30, 2009, 9:37:38 PM3/30/09
to Music Player Minion
* Added about:config option extensions.mpm.guess_tags, default=false.
Setting this to true will cause minion to guess the tags from the
filename when there is no Title tag from mpd, instead of just using
the filename.

* Fixed bug where playlist time is shown in seconds on startup,
instead
of hours, minutes, seconds.

* Fixed bug where queries and View Details did not work from the
statusbar playlist.

* Added mpdplaylist.crop() command, which can be used in a custom menu
item. Crop will delete all but the selected item(s) from the
playlist. I may add this to the default menu items in future
releases.

Klaus Schulz

unread,
Mar 31, 2009, 3:37:16 AM3/31/09
to music-pla...@googlegroups.com
Hi Chris.

Wow - Your .wav tag-guess works quite well on the first glance!!! Real great job. I am not aware of any other application under Linux doing this.

Comment: I had do insert the key to about:config manually.

I use this file-name-format: Tracknumber-Artist-Album-Tracktitle

This way I always have all information tight to a file-name without tags. All blanks are filled with underscores ( also a standard EAC feature - probably made for Unix folks)

Two early comments:

1. On my own playerscript (I use it to run ecasound)  I remove the ugly "underscores" when displaying the fields,  e.g. echo  "$1" | sed -e 's#_# #g'
    I am bringing this up because I'd guess that Unix folks (At least l am doing it) using the standard feature of  EAC to fill blanks with underscores.
     It would be really great to get a selectable option to swap "_" with " " in the tag-fields.
2. Are you using a fieldseparator to separate the fields? Perhaps it should be configurable, even though most of the times it should be "-".

Thx again for your great work.

Klaus

Chris Seickel

unread,
Mar 31, 2009, 12:28:56 PM3/31/09
to music-pla...@googlegroups.com
On Tue, Mar 31, 2009 at 3:37 AM, Klaus Schulz <kls....@googlemail.com> wrote:
Hi Chris.

Wow - Your .wav tag-guess works quite well on the first glance!!! Real great job. I am not aware of any other application under Linux doing this.

Thanks, I should have done this in the beginning.  I have quite a few untagged files in my collection and it is pretty annoying.  I should just go through and clean them up but I am too lazy. 
 

Comment: I had do insert the key to about:config manually.

I should have mentioned this.  The keys are not created until the first time they are used.  In this case it would be the first time you browse to a location that is not cached.  Speaking of caching, each unique database request is only made once and then the results are saved.  The cache will clear when you restart firefox or update mpd's database.


I use this file-name-format: Tracknumber-Artist-Album-Tracktitle

That is a very common format and will work perfectly with my tag guessing function.  Here is how it works:
1. Remove anything in parentheses from the filename, then split the filename into pieces based on the - separator.
2. Check each piece to see if it is a track number (numbers with or without a slash or period), remove it and assign it to the Track field.
3. The last piece is the Title, remove it.
4. If there are more pieces, the first is the Artist, remove it.
5. Anything left over is the Album.
 

This way I always have all information tight to a file-name without tags. All blanks are filled with underscores ( also a standard EAC feature - probably made for Unix folks)

Two early comments:

1. On my own playerscript (I use it to run ecasound)  I remove the ugly "underscores" when displaying the fields,  e.g. echo  "$1" | sed -e 's#_# #g'
    I am bringing this up because I'd guess that Unix folks (At least l am doing it) using the standard feature of  EAC to fill blanks with underscores.
     It would be really great to get a selectable option to swap "_" with " " in the tag-fields.

Done, the next release will replace underscores with spaces.
 

2. Are you using a fieldseparator to separate the fields? Perhaps it should be configurable, even though most of the times it should be "-".

I am using "-" as the separator.  I did a lot of experimentation with this a while back, and found that "-" is pretty universal as a separator.  If someone requests it, I can add in a custom separator option, but I don't see many people needing it.


Reply all
Reply to author
Forward
0 new messages