robotpy-pykit 0.2.1__tar.gz → 0.2.3__tar.gz

{robotpy_pykit-0.2.1 → robotpy_pykit-0.2.3}/LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2025 Westwood Robotics. All rights reserved.
+Copyright (c) 2025-2026 Westwood Robotics. All rights reserved.
 
 Redistribution and use in source and binary forms, with or without
 modification, are permitted provided that the following conditions are met:

{robotpy_pykit-0.2.1 → robotpy_pykit-0.2.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: robotpy-pykit
-Version: 0.2.1
+Version: 0.2.3
 Summary: A logging, telemetry, and replay framework for FRC robots running python
 Keywords: pykit,frc,robotics,logging,telemetry,replay
 Author: Luke Maxwell

{robotpy_pykit-0.2.1 → robotpy_pykit-0.2.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "uv_build"
 
 [project]
 name = "robotpy-pykit"
-version = "0.2.1"
+version = "0.2.3"
 description = "A logging, telemetry, and replay framework for FRC robots running python"
 readme = "README.md"
 authors = [{ name = "Luke Maxwell", email = "luke@whsrobotics.org" }]

{robotpy_pykit-0.2.1 → robotpy_pykit-0.2.3}/src/pykit/logger.py

@@ -269,7 +269,12 @@ class Logger:
                 rs = cls.replaySource
                 if rs is None or not rs.updateTable(cls.entry):
                     print("End of replay reached")
-                    cls.end()
+                    if cls.cycleCount == 1:
+                        print(
+                            "[ERROR] This robot did not start properly, is the replay logfile from PyKit?"
+                        )
+                    else:
+                        cls.end()
                     raise SystemExit(0)
 
             dsStart = RobotController.getFPGATime()

{robotpy_pykit-0.2.1 → robotpy_pykit-0.2.3}/src/pykit/wpilog/wpilogreader.py

@@ -5,6 +5,7 @@ from wpiutil.log import DataLogReader, DataLogRecord
 from pykit.logreplaysource import LogReplaySource
 from pykit.logtable import LogTable
 from pykit.logvalue import LogValue
+from pykit.wpilog import wpilogconstants
 
 T = TypeVar("T")
 
@@ -46,10 +47,8 @@ class WPILOGReader(LogReplaySource):
         self.reader = DataLogReader(self.filename)
         self.isValid = (
             self.reader.isValid()
-            # and self.reader.getExtraHeader() == wpilogconstants.extraHeader
+            and self.reader.getExtraHeader() == wpilogconstants.extraHeader
         )
-        print(self.reader.isValid())
-        print(self.reader.getExtraHeader())
         self.records = iter(())
 
         if self.isValid:
@@ -61,7 +60,10 @@ class WPILOGReader(LogReplaySource):
             self.entryCustomTypes: dict[int, str] = {}
 
         else:
-            print("[WPILogReader] not valid")
+            print(
+                "[WPILogReader] invalid data log!\n"
+                + "WPILogReader MUST use a WPILog generated with a WPILOGWriter"
+            )
 
     def updateTable(self, table: LogTable) -> bool:
         """

{robotpy_pykit-0.2.1 → robotpy_pykit-0.2.3}/src/pykit/wpilog/wpilogwriter.py

@@ -6,7 +6,8 @@ from tempfile import gettempdir
 from typing import TYPE_CHECKING
 
 from hal import MatchType
-from wpilib import DataLogManager, RobotBase, RobotController
+from wpilib import RobotBase, RobotController
+from wpiutil import DataLogWriter
 from pykit.logdatareciever import LogDataReciever
 from pykit.logger import Logger
 from pykit.logtable import LogTable
@@ -66,7 +67,7 @@ class WPILOGWriter(LogDataReciever):
             if filename is not None
             else f"pykit_{self.randomIdentifier}.wpilog"
         )
-        self.autoRename = False
+        self.autoRename = filename is None
 
     def start(self) -> None:
         """
@@ -82,16 +83,11 @@ class WPILOGWriter(LogDataReciever):
         if os.path.exists(fullPath):
             print("[WPILogWriter] File exists, overwriting")
             os.remove(fullPath)
-        if not RobotBase.isReal():
-            DataLogManager.stop()
         try:
-            DataLogManager.start(self.folder, self.filename)
+            self.log = DataLogWriter(fullPath, wpilogconstants.extraHeader)
         except PermissionError as e:
-            print(f"Failed to open WPILOG file! ({e})")
+            print(f"[WPILogWriter] Failed to open WPILOG file! ({e})")
             return
-        print(DataLogManager.getLogDir())
-        DataLogManager.logNetworkTables(False)
-        self.log = DataLogManager.getLog()
 
         self.isOpen = True
         self.timestampId = self.log.start(
@@ -106,7 +102,7 @@ class WPILOGWriter(LogDataReciever):
         self.entryTypes: dict[str, LogValue.LoggableType] = {}
         self.entryUnits: dict[str, str] = {}
         self.logDate = None
-        self.logMatchText = f"pykit_{self.randomIdentifier}"
+        self.logMatchText = ""
 
     def end(self) -> None:
         """
@@ -197,21 +193,9 @@ class WPILOGWriter(LogDataReciever):
                 if self.filename != filename:
                     # Rename log file by closing current and opening new
                     print(f"[WPILogWriter] Renaming log to {filename}")
-                    self.log.stop()
                     fullPath = os.path.join(self.folder, self.filename)
-                    if os.path.exists(fullPath):
-                        print(f"[WPILogWriter] Old file removed ({self.filename})")
-                        os.remove(fullPath)
-
-                    DataLogManager.stop()
-                    DataLogManager.start(self.folder, filename)
-                    self.log = DataLogManager.getLog()
-                    self.timestampId = self.log.start(
-                        "/Timestamp",
-                        LogValue.LoggableType.Integer.getWPILOGType(),
-                        wpilogconstants.entryMetadata,
-                        0,
-                    )
+                    os.rename(fullPath, os.path.join(self.folder, filename))
+
                     self.filename = filename
 
         # Write timestamp entry