From e84aabea81fd9efc874dc9f9d8f4e1dd543df3b6 Mon Sep 17 00:00:00 2001 From: Thibault Duplessis Date: Sat, 19 Oct 2013 11:50:06 +0200 Subject: [PATCH] drop analysis advices without variations --- modules/analyse/src/main/Analysis.scala | 4 +++- todo | 3 +-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/modules/analyse/src/main/Analysis.scala b/modules/analyse/src/main/Analysis.scala index 2b7c04399e..9d2ce759d0 100644 --- a/modules/analyse/src/main/Analysis.scala +++ b/modules/analyse/src/main/Analysis.scala @@ -11,7 +11,9 @@ case class Analysis( lazy val infoAdvices: InfoAdvices = { (Info.start :: infos) sliding 2 collect { - case List(prev, info) ⇒ info -> Advice(prev, info) + case List(prev, info) ⇒ info -> { + (old || info.hasVariation) ?? Advice(prev, info) + } } }.toList diff --git a/todo b/todo index 4cf8c06ca5..a973221024 100644 --- a/todo +++ b/todo @@ -73,8 +73,7 @@ no challenge when playing http://en.lichess.org/forum/lichess-feedback/defis#2 user option for auto-promotion (requested by NM :P) (http://en.lichess.org/@/Matetricks) check filter games icon on windows people page = trends by day, week, month. user achievments? -user options: tenth of seconds +user options: tenth of seconds, sounds analysis chart: title prevents clicking dots analysis chart: click dot does not go to move when on a variation -inacurracy without best move: http://en.lichess.org/forum/lichess-feedback/strange-analysis user stats: http://en.lichess.org/forum/lichess-feedback/feature-request-extended-stats