X-Git-Url: https://git.auder.net/?p=talweg.git;a=blobdiff_plain;f=pkg%2FR%2FF_Neighbors.R;fp=pkg%2FR%2FF_Neighbors.R;h=17fdd58d7a19b3e448e92f37e2e8d192a824e261;hp=9d1e3fbb84c8fb6e451e1ba08a3c57b7d6906d1f;hb=9b9bb2d490e356ff0393aec66b137cff0a64803d;hpb=882ae735725861e5a0f61b83556265dd367452dd

diff --git a/pkg/R/F_Neighbors.R b/pkg/R/F_Neighbors.R
index 9d1e3fb..17fdd58 100644
--- a/pkg/R/F_Neighbors.R
+++ b/pkg/R/F_Neighbors.R
@@ -128,8 +128,6 @@ NeighborsForecaster = R6::R6Class("NeighborsForecaster",
 				# limit=Inf to not censor any day (TODO: finite limit? 60?)
 				tdays = getSimilarDaysIndices(today, data, limit=Inf, same_season=TRUE,
 					days_in=tdays_cut, operational=opera)
-#				if (length(tdays) <= 1)
-#					return (NA)
 				# TODO: 10 == magic number
 				tdays = .getConstrainedNeighbs(today, data, tdays, min_neighbs=10)
 				if (length(tdays) == 1)
@@ -142,7 +140,7 @@ NeighborsForecaster = R6::R6Class("NeighborsForecaster",
 					}
 					return ( data$getSerie(tdays[1])[predict_from:horizon] )
 				}
-				max_neighbs = 10 #TODO: 12 = arbitrary number
+				max_neighbs = 10 #TODO: 10 or 12 or... ?
 				if (length(tdays) > max_neighbs)
 				{
 					distances2 <- .computeDistsEndo(data, today, tdays, predict_from)
@@ -226,11 +224,7 @@ NeighborsForecaster = R6::R6Class("NeighborsForecaster",
 .getConstrainedNeighbs = function(today, data, tdays, min_neighbs=10)
 {
 	levelToday = data$getLevelHat(today)
-#	levelYersteday = data$getLevel(today-1)
-	distances = sapply(tdays, function(i) {
-#		sqrt((data$getLevel(i-1)-levelYersteday)^2 + (data$getLevel(i)-levelToday)^2)
-		abs(data$getLevel(i)-levelToday)
-	})
+	distances = sapply( tdays, function(i) abs(data$getLevel(i) - levelToday) )
 	#TODO: 1, +1, +3 : magic numbers
 	dist_thresh = 1
 	min_neighbs = min(min_neighbs,length(tdays))
@@ -242,14 +236,7 @@ NeighborsForecaster = R6::R6Class("NeighborsForecaster",
 			break
 		dist_thresh = dist_thresh + ifelse(dist_thresh>1,3,1)
 	}
-	tdays = tdays[same_pollution]
-#	max_neighbs = 12
-#	if (nb_neighbs > max_neighbs)
-#	{
-#		# Keep only max_neighbs closest neighbors
-#		tdays = tdays[ order(distances[same_pollution])[1:max_neighbs] ]
-#	}
-	tdays
+	tdays[same_pollution]
 }
 
 # compute similarities
@@ -277,8 +264,7 @@ NeighborsForecaster = R6::R6Class("NeighborsForecaster",
 	sapply(tdays, function(i) {
 		delta = lastSerie - c(data$getSerie(i-1),
 			data$getSerie(i)[if (predict_from>=2) 1:(predict_from-1) else c()])
-#		sqrt(mean(delta^2))
-		sqrt(sum(delta^2))
+		sqrt(mean(delta^2))
 	})
 }