bugfixes (#16)

* display new windows per-callsign

* structure fields in grid

* check for network connectivity before sending packets

* variable reference fix

* logging

* add minimum time intervals to connections

* fix username setting

* test on macos

* job name

* use `psycopg2-binary`

* update README.md

* minimize windows on close

client/gui.py

@@ -121,6 +121,8 @@ def __init__(self, callsigns: [str] = None, log_filename: PathLike = None, outpu
         if self.output_filename is None:
             self.output_filename = Path('~') / 'Desktop'
 
+        self.__windows['main'].protocol("WM_DELETE_WINDOW", self.close)
+
         main_window.mainloop()
 
     @property
@@ -548,10 +550,17 @@ def retrieve_packets(self):
                     separator = Separator(window, orient=tkinter.VERTICAL)
                     separator.grid(row=0, column=3, rowspan=window.grid_size()[1] + 2, sticky='ns', padx=10)
 
+                    window.protocol("WM_DELETE_WINDOW", window.iconify)
+
                     self.__windows[callsign] = window
 
                 window = self.__windows[callsign]
 
+                if window.state() == 'iconic':
+                    window.deiconify()
+                if window.focus_get() is None:
+                    window.focus_force()
+
                 set_child_states(window)
 
                 packet_track = self.packet_tracks[callsign]
@@ -596,6 +605,11 @@ def replace_text(element: tkinter.Entry, value: str):
         element.delete(start_index, tkinter.END)
         element.insert(start_index, value)
 
+    def close(self):
+        if self.active:
+            self.toggle()
+        self.__windows['main'].destroy()
+
 
 def set_child_states(frame: tkinter.Frame, state: str = None, types: [type] = None):
     if state is None:

client/retrieve.py

@@ -48,7 +48,7 @@ def retrieve_packets(connections: [APRSPacketConnection], packet_tracks: [APRSTr
 
             if callsign not in packet_tracks:
                 packet_tracks[callsign] = APRSTrack(callsign, [parsed_packet])
-                logger.debug(f'started tracking {callsign:8}')
+                logger.debug(f'started tracking callsign {callsign:8}')
             else:
                 packet_track = packet_tracks[callsign]
                 if parsed_packet not in packet_track:

packetraven/connections.py

@@ -1,6 +1,5 @@
 from abc import ABC, abstractmethod
 from datetime import datetime, timedelta
-import logging
 from os import PathLike
 from pathlib import Path
 import re
@@ -247,7 +246,7 @@ def packets(self) -> [APRSPacket]:
             packets = [APRSPacket.from_raw_aprs(packet_candidate, source=self.location)
                        for packet_candidate in response['entries']]
         else:
-            logging.warning(f'query failure "{response["code"]}: {response["description"]}"')
+            LOGGER.warning(f'query failure "{response["code"]}: {response["description"]}"')
             packets = []
 
         self.__last_access_time = datetime.now()

packetraven/tracks.py

@@ -65,7 +65,7 @@ def time_to_ground(self) -> timedelta:
         if current_ascent_rate < 0:
             # TODO implement landing location as the intersection of the predicted descent track with a local DEM
             # TODO implement a time to impact calc based off of standard atmo
-            return self.packets[-1].coordinates[2] / current_ascent_rate
+            return timedelta(seconds=self.packets[-1].coordinates[2] / current_ascent_rate)
         else:
             return timedelta(seconds=-1)