Review Request: New dynamic bias

[Ralf Engels]

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


Reminder to myself:

We need a "if-else" Bias because currently an album-play is not possible.
So: if(pick next song from current album) else (pick song with track number 1)

To make the UI easier for non-experts:
Add three example default playlists
Add a "collection search bias" where you can add a search query from the collection search bar.

Current plan to improve the overview with a complex bias:
Use a tree view to display a textual representation of the bias. Only when clicked on should it display the full options in a popup or a dialog.

- Ralf


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/20110223/5baff80d/attachment-0001.htm