altos/altosui: Report log format in the version command
[fw/altos] / altosui / AltosEepromDownload.java
index f96a3dc90527559b685d2c28b94a090a53ba9467..417aab000f25158489ccc7255145117824029973 100644 (file)
@@ -89,90 +89,101 @@ public class AltosEepromDownload implements Runnable {
                }
        }
 
-       static final int        log_full = 1;
-       static final int        log_tiny = 2;
-
        boolean                 done;
        int                     state;
 
        void CaptureFull(AltosEepromChunk eechunk) throws IOException {
-               AltosEepromBlock        eeblock = new AltosEepromBlock(eechunk);
-
-               if (eeblock.has_flight) {
-                       flight = eeblock.flight;
-                       monitor.set_flight(flight);
-               }
-               if (eeblock.has_date) {
-                       year = eeblock.year;
-                       month = eeblock.month;
-                       day = eeblock.day;
-                       want_file = true;
-               }
+               boolean any_valid = false;
+               for (int i = 0; i < eechunk.chunk_size && !done; i += AltosEepromRecord.record_length) {
+                       try {
+                               AltosEepromRecord r = new AltosEepromRecord(eechunk, i);
+                               if (r.cmd == Altos.AO_LOG_FLIGHT) {
+                                       flight = r.b;
+                                       monitor.set_flight(flight);
+                               }
 
-               if (eeblock.size() == 0 ||
-                   eeblock.has_state && eeblock.state == Altos.ao_flight_landed)
-                       done = true;
+                               /* Monitor state transitions to update display */
+                               if (r.cmd == Altos.AO_LOG_STATE && r.a <= Altos.ao_flight_landed) {
+                                       state = r.a;
+                                       if (state > Altos.ao_flight_pad)
+                                               want_file = true;
+                               }
 
-               /* Monitor state transitions to update display */
-               if (eeblock.has_state) {
-                       if (eeblock.state > Altos.ao_flight_pad)
-                               want_file = true;
-                       if (eeblock.state > state)
-                               state = eeblock.state;
+                               if (r.cmd == Altos.AO_LOG_GPS_DATE) {
+                                       year = 2000 + (r.a & 0xff);
+                                       month = (r.a >> 8) & 0xff;
+                                       day = (r.b & 0xff);
+                                       want_file = true;
+                               }
+                               if (r.cmd == Altos.AO_LOG_STATE && r.a == Altos.ao_flight_landed)
+                                       done = true;
+                               any_valid = true;
+                               Log(r);
+                       } catch (ParseException pe) {
+                               if (parse_exception == null)
+                                       parse_exception = pe;
+                       }
                }
 
-               if (parse_exception == null && eeblock.parse_exception != null)
-                       parse_exception = eeblock.parse_exception;
+               if (!any_valid)
+                       done = true;
 
                CheckFile(false);
-
-               for (int record = 0; record < eeblock.size(); record++)
-                       Log(eeblock.get(record));
        }
 
        boolean start;
        int     tiny_tick;
 
        void CaptureTiny (AltosEepromChunk eechunk) throws IOException {
-               boolean some_reasonable_data = false;
+               boolean any_valid = false;
 
-               for (int i = 0; i < eechunk.data.length; i += 2) {
-                       int     v = eechunk.data16(i);
+               for (int i = 0; i < eechunk.data.length && !done; i += 2) {
+                       int                     v = eechunk.data16(i);
+                       AltosEepromRecord       r;
 
                        if (i == 0 && start) {
                                tiny_tick = 0;
                                start = false;
                                flight = v;
-                               Log(new AltosEepromRecord(Altos.AO_LOG_FLIGHT, tiny_tick, 0, v));
-                               some_reasonable_data = true;
+                               monitor.set_flight(flight);
+                               r = new AltosEepromRecord(Altos.AO_LOG_FLIGHT, tiny_tick, 0, v);
+                               any_valid = true;
                        } else {
                                int     s = v ^ 0x8000;
+
                                if (Altos.ao_flight_startup <= s && s <= Altos.ao_flight_invalid) {
-                                       Log(new AltosEepromRecord(Altos.AO_LOG_STATE, tiny_tick, s, 0));
-                                       if (s == Altos.ao_flight_landed) {
+                                       state = s;
+                                       r = new AltosEepromRecord(Altos.AO_LOG_STATE, tiny_tick, state, 0);
+                                       if (state == Altos.ao_flight_landed)
                                                done = true;
-                                               break;
-                                       }
-                                       some_reasonable_data = true;
+                                       state = s;
+                                       any_valid = true;
                                } else {
                                        if (v != 0xffff)
-                                               some_reasonable_data = true;
-                                       Log(new AltosEepromRecord(Altos.AO_LOG_HEIGHT, tiny_tick, v, 0));
+                                               any_valid = true;
+
+                                       r = new AltosEepromRecord(Altos.AO_LOG_PRESSURE, tiny_tick, 0, v);
+
+                                       /*
+                                        * The flight software records ascent data every 100ms, and descent
+                                        * data every 1s.
+                                        */
                                        if (state < Altos.ao_flight_drogue)
                                                tiny_tick += 10;
                                        else
                                                tiny_tick += 100;
                                }
                        }
+                       Log(r);
                }
                CheckFile(false);
-               if (!some_reasonable_data)
+               if (!any_valid)
                        done = true;
        }
 
        void CaptureLog(AltosEepromLog log) throws IOException, InterruptedException, TimeoutException {
                int                     block, state_block = 0;
-               int                     log_style = 0;
+               int                     log_format = flights.config_data.log_format;
 
                state = 0;
                done = false;
@@ -201,21 +212,24 @@ public class AltosEepromDownload implements Runnable {
                        AltosEepromChunk        eechunk = new AltosEepromChunk(serial_line, block);
 
                        /*
-                        * Figure out what kind of data is there
+                        * Guess what kind of data is there if the device
+                        * didn't tell us
                         */
 
-                       if (block == log.start_block) {
-                               if (eechunk.data(0) == Altos.AO_LOG_FLIGHT)
-                                       log_style = log_full;
-                               else
-                                       log_style = log_tiny;
+                       if (log_format == Altos.AO_LOG_FORMAT_UNKNOWN) {
+                               if (block == log.start_block) {
+                                       if (eechunk.data(0) == Altos.AO_LOG_FLIGHT)
+                                               log_format = Altos.AO_LOG_FORMAT_FULL;
+                                       else
+                                               log_format = Altos.AO_LOG_FORMAT_TINY;
+                               }
                        }
 
-                       switch (log_style) {
-                       case log_full:
+                       switch (log_format) {
+                       case Altos.AO_LOG_FORMAT_FULL:
                                CaptureFull(eechunk);
                                break;
-                       case log_tiny:
+                       case Altos.AO_LOG_FORMAT_TINY:
                                CaptureTiny(eechunk);
                                break;
                        }
@@ -250,11 +264,11 @@ public class AltosEepromDownload implements Runnable {
        }
 
        public void run () {
-               if (remote)
-                       serial_line.start_remote();
-
                try {
                        boolean failed = false;
+                       if (remote)
+                               serial_line.start_remote();
+
                        for (AltosEepromLog log : flights) {
                                parse_exception = null;
                                if (log.download) {
@@ -281,11 +295,12 @@ public class AltosEepromDownload implements Runnable {
                                                   serial_line.device.toShortString()),
                                     "Connection Failed",
                                     JOptionPane.ERROR_MESSAGE);
+               } finally {
+                       if (remote)
+                               serial_line.stop_remote();
+                       serial_line.flush_output();
                }
-               if (remote)
-                       serial_line.stop_remote();
                monitor.done();
-               serial_line.flush_output();
                if (listener != null) {
                        Runnable r = new Runnable() {
                                        public void run() {
@@ -317,6 +332,7 @@ public class AltosEepromDownload implements Runnable {
 
                frame = given_frame;
                serial_line = given_serial_line;
+               serial_line.set_frame(frame);
                remote = given_remote;
                flights = given_flights;
                success = false;