[Kde-games-devel] Review Request 109539: KNetWalk: New score calculation method

[Parker Coates]

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


If we want to switch to a new scoring algorithm, there's a lot more to it than just changing the code calculating the score.

First we'd need a method of dealing with the old highscores. Do we just drop them? Do we try to convert them to the new system? Is that even possible. Leaving them in place is not an option as it's a completely apples-to-oranges ranking.

Second, we should try to future proof the highscore logic so that it doesn't need to be changed again later. The big one I can think of is sub-second accuracy. Whole seconds are a pretty big unit to work with when comparing times, especially on Easy difficulty.

Third, any changes need to be reflected in the documentation.

I'm not trying to be pessimistic or imply that it can't be done. It just isn't a change to be taken too lightly.

- Parker Coates


On March 17, 2013, 1:52 p.m., Ashwin Rajeev wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109539/
> -----------------------------------------------------------
> 
> (Updated March 17, 2013, 1:52 p.m.)
> 
> 
> Review request for KDE Games.
> 
> 
> Description
> -------
> 
> Let the score be time taken to complete in seconds, that also means one with less score gets on top.
> Each penalty move will add 10 to score.
> 
> 
> Diffs
> -----
> 
>   src/mainwindow.cpp dc437fe 
> 
> Diff: http://git.reviewboard.kde.org/r/109539/diff/
> 
> 
> Testing
> -------
> 
> It may make problem with previously saved high scores.
> 
> 
> Thanks,
> 
> Ashwin Rajeev
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20130319/b4ab3ec9/attachment.html>