altoslib: Don't record 'pad' state in FlightSeries

[fw/altos] / altoslib / AltosTelemetry.java

diff --git a/altoslib/AltosTelemetry.java b/altoslib/AltosTelemetry.java
index 0ba9849a6c4c59fd24b0b1cdb3e2207dda82d716..b810ac83842715b0771337f7be625b7e62e37816 100644 (file)
--- a/altoslib/AltosTelemetry.java
+++ b/altoslib/AltosTelemetry.java
@@ -3,7 +3,8 @@
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful, but
  * WITHOUT ANY WARRANTY; without even the implied warranty of
@@ -15,7 +16,7 @@
  * 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA.
  */
 
-package org.altusmetrum.altoslib_9;
+package org.altusmetrum.altoslib_11;
 
 import java.text.*;
 
@@ -23,13 +24,16 @@ import java.text.*;
  * Telemetry data contents
  */
 
-public abstract class AltosTelemetry implements AltosStateUpdate {
+public abstract class AltosTelemetry implements AltosDataProvider {
+       int[]   bytes;
 
        /* All telemetry packets have these fields */
-       public int      tick;
-       public int      serial;
-       public int      rssi;
-       public int      status;
+       public int rssi() { return AltosConvert.telem_to_rssi(AltosLib.int8(bytes, bytes.length - 3)); }
+       public int status() { return AltosLib.uint8(bytes, bytes.length - 2); }
+
+       /* All telemetry packets report these fields in some form */
+       public abstract int serial();
+       public abstract int tick();
 
        /* Mark when we received the packet */
        long            received_time;
@@ -42,13 +46,14 @@ public abstract class AltosTelemetry implements AltosStateUpdate {
                return sum == bytes[bytes.length - 1];
        }
 
-       public void update_state(AltosState state) {
-               state.set_serial(serial);
-               if (state.state() == AltosLib.ao_flight_invalid)
-                       state.set_state(AltosLib.ao_flight_startup);
-               state.set_tick(tick);
-               state.set_rssi(rssi, status);
-               state.set_received_time(received_time);
+       public void provide_data(AltosDataListener listener, AltosCalData cal_data) {
+               cal_data.set_serial(serial());
+               if (listener.state == AltosLib.ao_flight_invalid)
+                       listener.set_state(AltosLib.ao_flight_startup);
+               cal_data.set_tick(tick());
+               listener.set_time(cal_data.time());
+               listener.set_rssi(rssi(), status());
+               listener.set_received_time(received_time);
        }
 
        final static int PKT_APPEND_STATUS_1_CRC_OK             = (1 << 7);
@@ -66,7 +71,8 @@ public abstract class AltosTelemetry implements AltosStateUpdate {
        final static int packet_type_mega_data = 0x09;
        final static int packet_type_metrum_sensor = 0x0a;
        final static int packet_type_metrum_data = 0x0b;
-       final static int packet_type_mini = 0x10;
+       final static int packet_type_mini2 = 0x10;
+       final static int packet_type_mini3 = 0x11;
 
        static AltosTelemetry parse_hex(String hex)  throws ParseException, AltosCRCException {
                AltosTelemetry  telem = null;
@@ -86,12 +92,6 @@ public abstract class AltosTelemetry implements AltosStateUpdate {
                if (!cksum(bytes))
                        throw new ParseException(String.format("invalid line \"%s\"", hex), 0);
 
-               int     rssi = AltosLib.int8(bytes, bytes.length - 3) / 2 - 74;
-               int     status = AltosLib.uint8(bytes, bytes.length - 2);
-
-               if ((status & PKT_APPEND_STATUS_1_CRC_OK) == 0)
-                       throw new AltosCRCException(rssi);
-
                /* length, data ..., rssi, status, checksum -- 4 bytes extra */
                switch (bytes.length) {
                case AltosLib.ao_telemetry_standard_len + 4:
@@ -106,35 +106,18 @@ public abstract class AltosTelemetry implements AltosStateUpdate {
                default:
                        throw new ParseException(String.format("Invalid packet length %d", bytes.length), 0);
                }
-               if (telem != null) {
-                       telem.received_time = System.currentTimeMillis();
-                       telem.rssi = rssi;
-                       telem.status = status;
-               }
                return telem;
        }
 
-       public static int extend_height(AltosState state, int height_16) {
-               double  compare_height;
-               int     height = height_16;
-
-               if (state.gps != null && state.gps.alt != AltosLib.MISSING) {
-                       compare_height = state.gps_height();
-               } else {
-                       compare_height = state.height();
-               }
-
-               if (compare_height != AltosLib.MISSING) {
-                       int     high_bits = (int) Math.floor (compare_height / 65536.0);
-
-                       height = (high_bits << 16) | (height_16 & 0xffff);
+       public AltosTelemetry() {
+               this.received_time = System.currentTimeMillis();
+       }
 
-                       if (Math.abs(height + 65536 - compare_height) < Math.abs(height - compare_height))
-                               height += 65536;
-                       else if (Math.abs(height - 65536 - compare_height) < Math.abs(height - compare_height))
-                               height -= 65536;
-               }
-               return height;
+       public AltosTelemetry(int[] bytes) throws AltosCRCException {
+               this();
+               this.bytes = bytes;
+               if ((status() & PKT_APPEND_STATUS_1_CRC_OK) == 0)
+                       throw new AltosCRCException(rssi());
        }
 
        public static AltosTelemetry parse(String line) throws ParseException, AltosCRCException {
@@ -147,7 +130,7 @@ public abstract class AltosTelemetry implements AltosStateUpdate {
                        throw new AltosCRCException(AltosParse.parse_int(word[i++]));
                }
 
-               AltosTelemetry telem;
+               AltosTelemetry telem = null;
 
                if (word[i].equals("TELEM")) {
                        telem = parse_hex(word[i+1]);