average wind speed was being miscalculated with a small number of sam…

…ples and UseSpeedForAvgCalc = true
cumulusmx · Dec 18, 2024 · 6751489 · 6751489
1 parent 4dc2d54
commit 6751489
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -37,6 +37,7 @@ Alternatively view it [online on GitHub](https://github.com/cumulusmx/CumulusMX/
 	- Databases migrated by v4.3.0 and v4.3.1 are now scanned for issues, and fixes automatically applied
 - Fix some errors parsing log file entries, affects load recent data, and graph files
 - Fix Unknown sensor errors in Ecowitt TCP API sensor info
+- Ecowitt historic catch-up average wind speed was being mishandled by MX = a lower value than reality
 
 
 ## [4.3.1 \[b4064\]][10] - 2024-12-09

diff --git a/CumulusMX/Properties/launchSettings.json b/CumulusMX/Properties/launchSettings.json
@@ -2,7 +2,7 @@
   "profiles": {
     "CumulusMX": {
       "commandName": "Project",
-      "commandLineArgs": "-lang en-GB -port 8998"
+      "commandLineArgs": "-lang fr-FR -port 8998"
     }
   }
 }
diff --git a/CumulusMX/WeatherStation.cs b/CumulusMX/WeatherStation.cs
@@ -6678,7 +6678,7 @@ public double GetWindAverageFromArray(DateTime fromTime)
 				}
 			}
 			// average the values, if we have enough samples
-			if (numvalues > 5)
+			if (numvalues > 5 || cumulus.StationOptions.UseSpeedForAvgCalc)
 			{
 				avg = totalwind / numvalues;
 			}