Review Request: New dynamic bias

Ralf Engels ralf-engels at gmx.de
Fri Feb 18 19:55:16 CET 2011



> On Feb. 13, 2011, 7:36 a.m., Sergey Ivanov wrote:
> > Some times I get several copies of 1 track in newer generated playlist (very rarely in random mode, and frequently with match by rating - my collection is very bad rated, so there are not so many tracks to choose from). I can't say is It appeared in this branch or It It's old bug (or not a bug at all), I'v never used dynamic playlists before. Also I can't understand how does "Quiz paly" works. Anyways 2nd day of tests and I haven't seen any errors, or bugs (except mentioned above).
> 
> Ralf Engels wrote:
>     Hi Sergey,
>     use the "unique" bias to filter out duplicate tracks.
>     
>     Quiz Play picks a track that starts with the character the last one ended with.
>     So after "Ronda" you get "All at once" and then "Every breath you take".
>     (you can also set this to use the artist instead of the track title)
>     
>     The Biases have tool tips.
> 
> Sergey Ivanov wrote:
>     Oh, thank you didn't know that. I just think that It shouldn't be separate mode, since usually users don't want to see duplicates in their playlist, especially a lot of copies of 1 track. I've got 10 copies of 1 song (entire playlist) when tried to use strict enough filter.
>     
>     Btw, got a crash when tried to abort Echonest request, but once again I'm completely not sure that It appeared in this branch.
>     http://pastebin.com/0SRUCCZw

I could reproduce the crash and fixed it.

Regarding Unique-songs.
I thought about it. I agree that usually you don't want to have duplicate tracks. However if you have a very selective Bias then duplicate tracks are the only correct result. However it should not end up like it was with the old Dynamic bias where it was quite likely that two tracks next to each other were duplicates.

For now I think that a separate "unique" bias is at least better than the old way where there was no way to prevent duplicates.


- Ralf


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100630/#review1400
-----------------------------------------------------------


On Feb. 11, 2011, 5:19 p.m., Ralf Engels wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100630/
> -----------------------------------------------------------
> 
> (Updated Feb. 11, 2011, 5:19 p.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Summary
> -------
> 
> This fix (actually not the attached patch but the dynamicplaylist branch) changes most of the dynamic playlist.
> 
> Main parts are the UI which was seen as confusing by many users and produced unexpected results.
> The global bias is now a "part" bias which allows you to set a part of the collection matching a criteria.
> The custom bias is now part of the bias system. I converted all existing custom biases to the new interface just to make sure that it's usable.
> The fuzzy bias was completely removed. If someone really wants to have a bias which results in a normal distribution of one numeric value then please feel free to add it again. For now you can define this kind of bias by defining something like this "(rating > 1) AND (rating < 4)"
> 
> The branch needs to be merged back to mainline and might have conflicts.
> 
> 
> This addresses bugs 175163, 175172, 177627, 188360, 188360, 228738, 230773, 232673, 233859, 260001, 260003, 265191, and, more, probably, and some.
>     https://bugs.kde.org/show_bug.cgi?id=175163
>     https://bugs.kde.org/show_bug.cgi?id=175172
>     https://bugs.kde.org/show_bug.cgi?id=177627
>     https://bugs.kde.org/show_bug.cgi?id=188360
>     https://bugs.kde.org/show_bug.cgi?id=188360
>     https://bugs.kde.org/show_bug.cgi?id=228738
>     https://bugs.kde.org/show_bug.cgi?id=230773
>     https://bugs.kde.org/show_bug.cgi?id=232673
>     https://bugs.kde.org/show_bug.cgi?id=233859
>     https://bugs.kde.org/show_bug.cgi?id=260001
>     https://bugs.kde.org/show_bug.cgi?id=260003
>     https://bugs.kde.org/show_bug.cgi?id=265191
>     https://bugs.kde.org/show_bug.cgi?id=and
>     https://bugs.kde.org/show_bug.cgi?id=more
>     https://bugs.kde.org/show_bug.cgi?id=probably
>     https://bugs.kde.org/show_bug.cgi?id=some
> 
> 
> Diffs
> -----
> 
>   ChangeLog 809d5e7 
> 
> Diff: http://git.reviewboard.kde.org/r/100630/diff
> 
> 
> Testing
> -------
> 
> Used it for a couple of times and wrote new auto tests.
> Tested against bug reports.
> 
> 
> Thanks,
> 
> Ralf
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/amarok-devel/attachments/20110218/04d15e44/attachment.htm 


More information about the Amarok-devel mailing list