Fix telemetry when clock is 0

arduino/tempLightSensor/tempLightSensor.ino

@@ -22,6 +22,7 @@ int photocellPin = A1;
 
 EthernetUDP ntpUDP;
 NTPClient timeClient(ntpUDP);
+bool NTPValid = false;
 
 struct netConfig {
   IPAddress address;
@@ -71,7 +72,9 @@ void setup(void) {
   timeClient.setPoolServerIP(config.address);
 #endif
   timeClient.begin();
-  timeClient.update();
+  if (timeClient.update()) {
+    NTPValid = true;
+  }
 
 #if DEBUG_TO_SERIAL
   Serial.println("DEBUG: clock updated via NTP.");
@@ -89,7 +92,11 @@ void loop(void) {
   float tempVoltage = tempReading * AREF_VOLTAGE / 1024.0;
   float tempC = (tempVoltage - 0.5) * 100 ;
 
-  json["time"] = timeClient.getEpochTime();
+  if (NTPValid) {
+    json["clock"] = timeClient.getEpochTime();
+  } else {
+    json["clock"] = NULL; // converted into 0
+  }
   payload["light"] = photocellReading;
 
   temp["celsius"] = tempC;

freedcs/api/admin.py

@@ -5,6 +5,8 @@ from api.models import Device, WhiteList
 @admin.register(Device)
 class DeviceAdmin(admin.ModelAdmin):
     readonly_fields = ('creation_time', 'updated_time',)
+    list_filter = ('serial',)
+    search_fields = ('serial',)
 
     fieldsets = (
         (None, {
@@ -20,6 +22,8 @@ class DeviceAdmin(admin.ModelAdmin):
 @admin.register(WhiteList)
 class WhiteListAdmin(admin.ModelAdmin):
     readonly_fields = ('creation_time', 'updated_time',)
+    list_filter = ('serial',)
+    search_fields = ('serial',)
 
     fieldsets = (
         (None, {

freedcs/telemetry/admin.py

@@ -4,4 +4,7 @@ from telemetry.models import Telemetry
 
 @admin.register(Telemetry)
 class TelemetryAdmin(admin.ModelAdmin):
-    readonly_fields = ('device', 'time', 'payload',)
+    readonly_fields = ('device', 'time', 'clock', 'payload',)
+    list_display = ('__str__', 'device')
+    list_filter = ('time', 'device__serial')
+    search_fields = ('device__serial',)

freedcs/telemetry/migrations/0005_telemetry_clock.py

@@ -0,0 +1,19 @@
+# Generated by Django 3.0.6 on 2020-06-03 13:08
+
+import django.core.validators
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('telemetry', '0004_auto_20200602_2132'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='telemetry',
+            name='clock',
+            field=models.IntegerField(null=True, validators=[django.core.validators.MinValueValidator(0)]),
+        ),
+    ]

freedcs/telemetry/migrations/0006_auto_20200603_1317.py

@@ -0,0 +1,17 @@
+# Generated by Django 3.0.6 on 2020-06-03 13:17
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('telemetry', '0005_telemetry_clock'),
+    ]
+
+    operations = [
+        migrations.AlterModelOptions(
+            name='telemetry',
+            options={'ordering': ['-time', 'device'], 'verbose_name_plural': 'Telemetry'},
+        ),
+    ]

freedcs/telemetry/models.py

@@ -1,4 +1,5 @@
 from django.db import models
+from django.core.validators import MinValueValidator
 from django.contrib.postgres.fields import JSONField
 
 from api.models import Device
@@ -7,11 +8,14 @@ from api.models import Device
 class Telemetry(models.Model):
     device = models.ForeignKey(Device, on_delete=models.CASCADE)
     time = models.DateTimeField(primary_key=True, auto_now_add=True)
+    clock = models.IntegerField(
+        validators=[MinValueValidator(0)],
+        null=True)
     payload = JSONField()
 
     class Meta:
-        ordering = ['time', 'device']
+        ordering = ['-time', 'device']
         verbose_name_plural = "Telemetry"
 
     def __str__(self):
-        return "%s - %s" % (self.time, self.device.serial)
+        return str(self.time)

freedcs/telemetry/serializers.py

@@ -11,7 +11,7 @@ class TelemetrySerializer(serializers.ModelSerializer):
 
     class Meta:
         model = Telemetry
-        fields = ('device', 'time', 'payload',)
+        fields = ('time', 'device', 'clock', 'payload',)
 
 
 # class WhiteListSerializer(serializers.ModelSerializer):