View Issue Details

IDProjectCategoryView StatusLast Update
0001656MMW v4Properties/Auto-Toolspublic2005-05-09 14:07
Reporterrusty Assigned To 
PriorityhighSeverityfeatureReproducibilityalways
Status feedbackResolutionreopened 
Summary0001656: Basic Auto-tagging functionality via Album Browser (Amazon window)
DescriptionThis is to track some possible minor improvements to the Album browser that would allow it to use the metadata contained within for the purposes of updating track metadata. It's not a complete solution to 0000085, but it may be a quick and dirty solution that we could implement in the short term (i.e. for 2.3).

High level goals:
1) Provide ability to selectively update track properties for the Currently Playing track, based on metadata/image in the album browser
2) Nice to have: Ability to update track properties for other tracks on the same album as that currently being played
3) The implementation should be lightweight, since it's very possible that it will be thrown away when we build a more comprehensive solution.

Possible implementation:
1) In the Album Browser each item would have a checkbox next to it, and there'd be a button at the bottom of the dialog called 'Update Properties'.

2) There are 2 possible approaches to updating other tracks on the album:
a) Add the following to the bottom of the dialog:
   [ ] Apply album properties to all tracks on the Album

b) Another slightly more advanced approach could work as follows:
   [ ] Apply to all tracks on the Album

If enabled, the window would extend down - the list of tracks from album would appear there.
 - User could reorder the tracks there (d&d), but they would most likely be properly ordered by MM because it would either use track #s or some AI to map track titles with those from Amazon.

c) An even more advanced approach would have the logic described in b) be applicable to any set of selected tracks (rather than just the currently playing track).
Additional InformationBug 0000085 describes the full set of required functionality (rather than a short term approach).
Similar implementations:
http://www.softpointer.com/tr_screenshots.htm (last screencap)

Useful Album art only tools:
http://str8dog.com/archive/2004/06/23/1232.aspx
http://louhi.kempele.fi/~skyostil/projects/albumart/
TagsNo tags attached.
Fixed in build835

Relationships

related to 0010616 closedLudek Lyrics lookup 

Activities

rusty

2004-12-16 06:12

administrator   ~0004947

Last edited: 2004-12-17 06:11

Comments updated 12/16 based on testing on build 820 (I'm not entering individual bugs since these are probably all issues that you might be aware of).
a) This dialog can be called 'Album Art & Info Lookup'.
b) It should be accessible via menu/context commands called 'Lookup Album Art & Info'. (Note: 'Get Track Details' on the context menu should be removed).
c) View|Album Browser can also be eliminated (i.e. the Album browser is no longer required).
d) This dialog should support a mode of operation in which the user can update tracks as he/she is listening to them. In build 820, this doesn't work (i.e. Lock to Now Playing has no effect).
e) The dialog should allow the user to access other portions of MediaMonkey--especially if Lock to Now Playing is being used. This allows the user to compare existing properties with the proposed properties.
f) It would be nice if there was a way to deselect all Track info at once, but retain Album Art. I make this point because very often, all I want to import is the Album Art, but I'm forced to uncheck all the other attributes one by one.
g) The functionality doesn't seem to work for OGG files in build 820 (the image appears, but after closing the dialog, it disappears).
h) There's no way to apply Album Art to all tracks on the Album in the event that the user didn't select all tracks. I don't think this is really a major issue, but it could be fixed by asking user regarding whether they want all other tracks with the same album name to be updated.
i) Album Art selection location needs to be added (per IM discussion).

jiri

2004-12-18 21:30

administrator   ~0004959

Fixed in build 821.

a&b) I used other strings you suggested in an e-mail, we can discuss further.
d) It doesn't seem really possible for several reasons (not big effectivity, undo problems, etc.). We can discuss.
e) I think that this will be solved by adding some more features to the dialog (playback of files, editing their properties, etc.)
f) Do you mean not to apply track names? You can deselect 'Tracks:' checked box only.
g) It stored directly to tags only, so OGG are only supported now when images can be linked to files.
h) That's quite a problem since MM won't often know what the album is (not album and album artists filled properly).
i) Added.

Also because some images are GIFs in Amazon collection I added support for them (so now JPG, PNG, BMP and GIF are supported types of files and tag storages).

rusty

2004-12-19 14:17

administrator   ~0004961

Last edited: 2004-12-21 22:01

Looking very nice, though there a few things I think we should resolve:

d)&e) As discussed it would be useful if:
i) the dialog had '<' and '>' arrows similar to the Properties dialog (IM: the arrows would only be active if there are unselected tracks prior/after the current selection)
ii) the dialog functioned similarly to the View|Properties dialog (i.e. the user can go in and out of it) (Jiri IM: this may be problematic due to the embedded IE control/Window on top issues)
iii) the dialog could be 'locked' to the curretly playing track (like View|Album Browser) (with the caveat that changes would not be committed if a song changed and undo would not work).

j) (new issue)- The current implementation is very useful when the user knows that a particular set of tracks are part of an album, but when he is trying to lookup tracks one by one (because he does not know what albums they belong to), the results that are returned are too broad and not in chronological order. Is there anything we can do about this? One thing I can think of is to at least highlight tracks that contain the track in some manner (e.g. Bold). Another is that if Artist=Sweet, at least list entries where 'sweet' is part of the Album Title after all entries where Artist=Sweet (or at least, sort the entries by 1) Artist 2) year).

{IM discussion: Jiri indicated that we _must_ show based on _relevance_, however, it seems to me that our guesses of relevance aren't good when the user doesn't know what tracks are part of an album and i)uses the functionality with a set of tracks by one artist that isn't part of a single album ii)uses the functionality with a single track. An alternative would be to allow the user to sort the suggested Albums by any number of fields).

k) (new issue) - When the user is looking up metadata for a track/tracks and isn't satisfied with the initial recommendation, and tries switching to an alternate album (via the drop down list), the Track #s don't update (so the user doesn't have immediate feedback as to whether the new album is good or not--e.g. if it is bad, then '?' should appear for the track#).

l) (new issue) - The track# column should look different from the other columns (it isn't a simple track property in the same way as the other columns are.

m) (new issue)- The dialog that appears every time prompting the user to choose a tag location is annoying and needlessly repetitive. Also the filenames should be configurable via a mask so that the user doesn't have to enter them over and over.

n) (new issue) - Sometimes it seems as if the information from Amazon never stops coming and the user has no idea as to when the info from Amazon will be complete. MM should have a 'Downloading: x%' indicator (similar to moreTunes). Also, it should probably limit the number of downloads (somehow moreTunes seems to download Albums much more quickly).

o) (new issue) - If the user does a search for a track that is from the same Album as other tracks in the library, it would be a good idea to include those other tracks in the dialog, but not check them off. That way, the user could easily apply updates to all tracks from the Album. This is to meet the usecase requested at: http://www.mediamonkey.com/forum/viewtopic.php?t=2982

p) (new issue) - If the user just wants to update album art, it's a pain to unselect the other fields each time. Suggested logic: Fields should be pre-checked only if: 1) they are found in the lookup 2) They are empty for at least one of the tracks that are being looked up. This would prevent 're-updating' of existing information with incorrect data.

Alternate solution1: Allow the user to configure, via Options, which fields to enable OR default to the logic described above.

jiri

2004-12-20 10:27

administrator   ~0004970

m) Was fixed in build 823 by letting user to prevent further showing of the dialog and also chosing the way of tagging by pressing Options button in Auto-tag dialog.

rusty

2004-12-21 21:41

administrator   ~0004976

Note: retest k) (jiri indicated it's not reproducible.

jiri

2005-01-03 09:04

administrator   ~0005086

Fixed in build 828.
 d)ii) Modified.
 l) Font changed to bold.
Other issues either don't seem to be really issues to me or can wait. Just:
 n) I think that an animated AVI would be best for this (i.e. as in IE). It can probably wait for now, but if we can get some nice animation (best of all e.g. our monkey) it could be in top right corner.

rusty

2005-01-04 05:51

administrator   ~0005102

Tested 828:

dii) works but I think in an unintended way. It keeps opening more and more Auto-Tag dialogs (e.g. after editing 10 albums, I have 10 dialogs open)!

l) ok

n) I doubt we'll get this for 2.3, but we'll see

jiri

2005-01-04 08:08

administrator   ~0005105

dii) Actually, I planned it this way. I.e. user can close the window by Close button, can access all MM functions and also can open another tagging window e.g. for comparing album covers, tag several albums at once or so. We can discuss over IM...

rusty

2005-01-12 13:03

administrator   ~0005151

Triging to 'Urgent'. Remaining issues can be specced/triaged/split into new bugs during the next release cycle.

rusty

2005-01-14 16:37

administrator   ~0005161

Re-prioritizing to 'immediate' in order fix issue dii) which has been raised by users in several posts:

http://www.mediamonkey.com/forum/viewtopic.php?t=3198
http://www.mediamonkey.com/forum/viewtopic.php?t=3176

jiri

2005-01-23 11:17

administrator   ~0005201

Fixed in build 835.

rusty

2005-01-31 16:35

administrator   ~0005237

Verified in 835. Works as expected.

Re-opening as 'Urgent' to document any remaining issues for future releases.