From 4761716bea72bcffdac3348ec2cb6e6cbaef7a14 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jos=C3=A9=20Rebelo?= <joserebelo@outlook.com>
Date: Sat, 16 May 2026 18:47:38 +0100
Subject: [PATCH] Health Connect: Move per-sample logging to TRACE

---
 .../gadgetbridge/devices/AbstractSampleProvider.java            | 2 +-
 .../util/healthconnect/syncers/AbstractActivitySampleSyncer.kt  | 2 +-
 .../util/healthconnect/syncers/AbstractTimeSampleSyncer.kt      | 2 +-
 .../util/healthconnect/syncers/TemperatureSyncer.kt             | 2 +-
 .../gadgetbridge/util/healthconnect/syncers/Vo2MaxSyncer.kt     | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/AbstractSampleProvider.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/AbstractSampleProvider.java
index c4ad2d8ea9..6e39fbe01f 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/AbstractSampleProvider.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/devices/AbstractSampleProvider.java
@@ -404,7 +404,7 @@ public abstract class AbstractSampleProvider<T extends AbstractActivitySample> i
         T previousSample = it.next();
 
         if (LOG.isTraceEnabled()) {
-            LOG.trace("Starting filling gapps at {}", DateTimeUtils.formatDateTime(DateTimeUtils.parseTimestampMillis(previousSample.getTimestamp() * 1000L)));
+            LOG.trace("Starting filling gaps at {}", DateTimeUtils.formatDateTime(DateTimeUtils.parseTimestampMillis(previousSample.getTimestamp() * 1000L)));
         }
 
         while (it.hasNext()) {
diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/AbstractActivitySampleSyncer.kt b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/AbstractActivitySampleSyncer.kt
index a31e015365..2254969258 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/AbstractActivitySampleSyncer.kt
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/AbstractActivitySampleSyncer.kt
@@ -77,7 +77,7 @@ internal abstract class AbstractActivitySampleSyncer<TRecord : Record> : Activit
             val startTs = endTs.minus(1, ChronoUnit.MINUTES)
 
             if (endTs.isBefore(sliceStartBoundary) || startTs.isAfter(sliceEndBoundary)) {
-                logger.debug(
+                logger.trace(
                     "Skipping {} for device '{}' for sample at {} (interval {} to {}) as its interval is outside the slice {} - {}.",
                     recordTypeName,
                     deviceName,
diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/AbstractTimeSampleSyncer.kt b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/AbstractTimeSampleSyncer.kt
index 4df673700f..06fbfb27d4 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/AbstractTimeSampleSyncer.kt
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/AbstractTimeSampleSyncer.kt
@@ -105,7 +105,7 @@ internal abstract class AbstractTimeSampleSyncer<TSample : TimeSample, TRecord :
         val recordsToInsert = samples.filter {
             val timestamp = Instant.ofEpochMilli(it.timestamp)
             if (timestamp.isBefore(sliceStartBoundary) || timestamp.isAfter(sliceEndBoundary)) {
-                logger.debug(
+                logger.trace(
                     "Skipping sample for at {} as it's outside the slice {} - {}.",
                     timestamp,
                     sliceStartBoundary,
diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/TemperatureSyncer.kt b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/TemperatureSyncer.kt
index bf3bfce0c9..596af5b33c 100755
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/TemperatureSyncer.kt
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/TemperatureSyncer.kt
@@ -115,7 +115,7 @@ internal object TemperatureSyncer : HealthConnectSyncer {
                         )
                     )
                 } else {
-                    LOG.debug(
+                    LOG.trace(
                         "Skipping Body Temperature sample for device '{}' at {} (value: {}°C) as it's outside slice {} - {}.",
                         deviceName,
                         timestamp,
diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/Vo2MaxSyncer.kt b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/Vo2MaxSyncer.kt
index dc7ac14b13..b854a4e7a1 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/Vo2MaxSyncer.kt
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/healthconnect/syncers/Vo2MaxSyncer.kt
@@ -46,7 +46,7 @@ internal object Vo2MaxSyncer : AbstractTimeSampleSyncer<Vo2MaxSample, Vo2MaxReco
         deviceName: String
     ): Vo2MaxRecord? {
         if (sample.value <= 0) {
-            logger.debug("Skipping VO2Max sample for device '$deviceName' due to invalid value: ${sample.value}.")
+            logger.trace("Skipping VO2Max sample for device '$deviceName' due to invalid value: ${sample.value}.")
             return null
         }