From dec0e7d5b22530cedb038c8f647e677df72505a7 Mon Sep 17 00:00:00 2001
From: RenatusRo <126881912+RenatusRo@users.noreply.github.com>
Date: Wed, 8 Apr 2026 12:35:17 +0200
Subject: [PATCH] Optimizer: improve error message for solar forecast (#28934)

---
 core/site_optimizer.go | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/core/site_optimizer.go b/core/site_optimizer.go
index b351a9596..8ce7eb9b0 100644
--- a/core/site_optimizer.go
+++ b/core/site_optimizer.go
@@ -136,12 +136,16 @@ func (site *Site) optimizerUpdate(battery []types.Measurement) error {
 	feedIn := currentRates(site.GetTariff(api.TariffUsageFeedIn))
 
 	minLen := lo.Min([]int{len(grid), len(feedIn)})
-	if solarTariff != nil {
-		// allow empty solar forecast
+	// exclude empty solar forecast from minLen
+	if solarTariff != nil && len(solar) > 0 {
 		minLen = min(minLen, len(solar))
 	}
-	if minLen < 8 {
-		return fmt.Errorf("not enough slots for optimization: %d (grid=%d, feedIn=%d, solar=%d)", minLen, len(grid), len(feedIn), len(solar))
+	const expectedSlots = 8
+	if minLen < expectedSlots {
+		if solarTariff != nil {
+			return fmt.Errorf("not enough forecast slots for meaningful optimization: %d < %d (grid=%d, feedIn=%d, solar=%d)", minLen, expectedSlots, len(grid), len(feedIn), len(solar))
+		}
+		return fmt.Errorf("not enough forecast slots for meaningful optimization: %d < %d (grid=%d, feedIn=%d)", minLen, expectedSlots, len(grid), len(feedIn))
 	}
 
 	now := time.Now()