another fix for tx send

aprsd/packets/core.py

@@ -102,7 +102,6 @@ class Packet:
     send_count: int = field(repr=False, default=0, compare=False, hash=False)
     retry_count: int = field(repr=False, default=3, compare=False, hash=False)
     last_send_time: float = field(repr=False, default=0, compare=False, hash=False)
-    last_send_attempt: int = field(repr=False, default=0, compare=False, hash=False)
 
     # Do we allow this packet to be saved to send later?
     allow_delay: bool = field(repr=False, default=True, compare=False, hash=False)

aprsd/packets/packet_list.py

@@ -18,12 +18,13 @@ class PacketList(objectstore.ObjectStoreMixin):
     lock = threading.Lock()
     _total_rx: int = 0
     _total_tx: int = 0
+    _maxlen: int = 100
 
     def __new__(cls, *args, **kwargs):
         if cls._instance is None:
             cls._instance = super().__new__(cls)
-            cls._maxlen = CONF.packet_list_maxlen
+            cls._instance._maxlen = CONF.packet_list_maxlen
-            cls.data = {
+            cls._instance.data = {
                 "types": {},
                 "packets": OrderedDict(),
             }
@@ -58,7 +59,7 @@ class PacketList(objectstore.ObjectStoreMixin):
     def _add(self, packet):
         if packet.key in self.data["packets"]:
             self.data["packets"].move_to_end(packet.key)
-        elif len(self.data["packets"]) == self.maxlen:
+        elif len(self.data["packets"]) == self._maxlen:
             self.data["packets"].popitem(last=False)
         self.data["packets"][packet.key] = packet
 

aprsd/packets/seen_list.py

@@ -26,10 +26,16 @@ class SeenList(objectstore.ObjectStoreMixin):
             cls._instance.data = {}
         return cls._instance
 
+    @wrapt.synchronized(lock)
     def stats(self, serializable=False):
         """Return the stats for the PacketTrack class."""
         return self.data
 
+    @wrapt.synchronized(lock)
+    def copy(self):
+        """Return a copy of the data."""
+        return self.data.copy()
+
     @wrapt.synchronized(lock)
     def update_seen(self, packet):
         callsign = None

aprsd/packets/tracker.py

@@ -65,10 +65,9 @@ class PacketTrack(objectstore.ObjectStoreMixin):
         pkts = {}
         for key in self.data:
             last_send_time = self.data[key].last_send_time
-            last_send_attempt = self.data[key]._last_send_attempt
             pkts[key] = {
                 "last_send_time": last_send_time,
-                "last_send_attempt": last_send_attempt,
+                "send_count": self.data[key].send_count,
                 "retry_count": self.data[key].retry_count,
                 "message": self.data[key].raw,
             }
@@ -82,7 +81,7 @@ class PacketTrack(objectstore.ObjectStoreMixin):
     @wrapt.synchronized(lock)
     def add(self, packet):
         key = packet.msgNo
-        packet._last_send_attempt = 0
+        packet.send_count = 0
         self.data[key] = packet
         self.total_tracked += 1
 

aprsd/threads/tx.py

@@ -118,7 +118,7 @@ class SendPacketThread(aprsd_threads.APRSDThread):
             return False
         else:
             send_now = False
-            if packet.send_count == packet.retry_count:
+            if packet.send_count >= packet.retry_count:
                 # we reached the send limit, don't send again
                 # TODO(hemna) - Need to put this in a delayed queue?
                 LOG.info(