Bug 170829 - Collection scan : large compilations aren't recognized as compilations
Summary: Collection scan : large compilations aren't recognized as compilations
Status: RESOLVED DUPLICATE of bug 172953
Alias: None
Product: amarok
Classification: Applications
Component: Collections/Local (show other bugs)
Version: 2.0-SVN
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Amarok Developers
URL:
Keywords: needs_verification
Depends on:
Blocks:
 
Reported: 2008-09-10 23:37 UTC by Carl
Modified: 2008-10-16 14:58 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Carl 2008-09-10 23:37:23 UTC
Version:           1.90 (using KDE 4.1.1)
OS:                Linux
Installed from:    Ubuntu Packages

I have a few large compilations (between 500 and 900 tracks each).
When amarok scans my collection it creates one album per track for these compilation, instead of one.
It detects smaller compilations (20-30 tracks) correctly.
It works well with all size compilations in amarok 1.

After reading a bit the code, it seems to come from the ScanResultProcessor.cpp file, processDirectory function :
//using the following heuristics:
//if more than one album is in the dir, use the artist of each track as albumartist
//if more than 60 files are in the dir, use the artist of each track as albumartist
//if all tracks have the same artist, use it as albumartist
//try to find the albumartist A: tracks must have the artist A or A feat. B (and variants)
//if no albumartist could be found, it's a compilation

The problem may come from this 60 limit.
Why 60 ? Could it be a higher number (maybe 1000) ? or this constraint could disappear ? or become customizable ?
Comment 1 Mark Kretschmann 2008-09-11 07:54:01 UTC
I can confirm that this also happens with one collection we have, "100 MTV Number one Hits" or somesuch. And it is _very_ annoying.
Comment 2 Mark Kretschmann 2008-09-11 08:01:04 UTC
I should add that I was referring to Ampache. Dunno if that makes a difference.
Comment 3 Mark Kretschmann 2008-10-16 14:58:24 UTC

*** This bug has been marked as a duplicate of bug 172953 ***