trajectopy 4.3.0__tar.gz → 4.3.1__tar.gz

{trajectopy-4.3.0 → trajectopy-4.3.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: trajectopy
-Version: 4.3.0
+Version: 4.3.1
 Summary: Trajectory Evaluation in Python
 Project-URL: Homepage, https://gereon-t.github.io/trajectopy/
 Project-URL: Documentation, https://gereon-t.github.io/trajectopy/

{trajectopy-4.3.0 → trajectopy-4.3.1}/example_data/report_settings.json

@@ -19,9 +19,9 @@
     "scatter_smooth": false,
     "scatter_smooth_window": 5,
     "scatter_plot_on_map": false,
-    "scatter_mapbox_style": "open-street-map",
+    "scatter_mapbox_style": "carto-positron",
     "scatter_mapbox_zoom": 15,
-    "scatter_mapbox_token": "pk.eyJ1IjoiZ2VyZW9uLXQiLCJhIjoiY2xwdGp6Nnh5MGZlejJqcnptemdlZnJicyJ9.fbQAv2C5pw7ctmzP_qX1pg",
+    "scatter_mapbox_token": "",
     "pos_x_name": "x",
     "pos_y_name": "y",
     "pos_z_name": "z",

{trajectopy-4.3.0 → trajectopy-4.3.1}/test/test_merging.py

@@ -59,7 +59,8 @@ class TestMerging(unittest.TestCase):
         self.assertIsInstance(merged, Trajectory)
 
         # Verify length is sum of input trajectories
-        expected_length = len(self.traj1) + len(self.traj2)
+        unique_timestamps = np.unique(np.concatenate([self.traj1.timestamps, self.traj2.timestamps]))
+        expected_length = len(unique_timestamps)
         self.assertEqual(len(merged), expected_length)
 
         # Verify name
@@ -81,7 +82,10 @@ class TestMerging(unittest.TestCase):
         merged = merge_trajectories([self.traj1, self.traj2, self.traj3])
 
         # Verify all trajectories are included
-        expected_length = len(self.traj1) + len(self.traj2) + len(self.traj3)
+        unique_timestamps = np.unique(
+            np.concatenate([self.traj1.timestamps, self.traj2.timestamps, self.traj3.timestamps])
+        )
+        expected_length = len(unique_timestamps)
         self.assertEqual(len(merged), expected_length)
 
         # Timestamps should still be sorted
@@ -104,9 +108,11 @@ class TestMerging(unittest.TestCase):
 
         merged = merge_trajectories([traj1_no_rot, traj2_no_rot])
 
+        unique_timestamps = np.unique(np.concatenate([traj1_no_rot.timestamps, traj2_no_rot.timestamps]))
+
         # Should work without rotations (creates identity rotations)
         self.assertIsNotNone(merged)
-        self.assertEqual(len(merged), len(traj1_no_rot) + len(traj2_no_rot))
+        self.assertEqual(len(merged), len(unique_timestamps))
 
     def test_merge_trajectories_preserves_epsg(self):
         """Test that EPSG code is preserved in merging."""

trajectopy-4.3.1/trajectopy/__version__.py

@@ -0,0 +1 @@
+__version__ = "4.3.1"

{trajectopy-4.3.0 → trajectopy-4.3.1}/trajectopy/core/settings.py

@@ -980,7 +980,7 @@ class ReportSettings(Settings):
         scatter_smooth: Smooth scatter plot color data. Defaults to False.
         scatter_smooth_window: Window size for scatter smoothing. Defaults to 5.
         scatter_plot_on_map: Plot trajectories on mapbox map (requires valid EPSG). Defaults to False.
-        scatter_mapbox_style: Mapbox style (some require token). Defaults to "open-street-map".
+        scatter_mapbox_style: Mapbox style (some require token). Defaults to "carto-positron".
         scatter_mapbox_zoom: Mapbox zoom level. Defaults to 15.
         scatter_mapbox_token: Mapbox access token (optional). Defaults to "".
         pos_x_name: Label for X position coordinate. Defaults to "x".
@@ -1026,7 +1026,7 @@ class ReportSettings(Settings):
     scatter_smooth_window: int = 5
 
     scatter_plot_on_map: bool = False
-    scatter_mapbox_style: str = "open-street-map"
+    scatter_mapbox_style: str = "carto-positron"
     scatter_mapbox_zoom: int = 15
     scatter_mapbox_token: str = ""
 

{trajectopy-4.3.0 → trajectopy-4.3.1}/trajectopy/core/trajectory.py

@@ -73,6 +73,20 @@ class Trajectory:
 
         self.timestamps = np.arange(0, len(positions)) if timestamps is None else timestamps
 
+        # ensure strictly increasing timestamps
+        unique_timestamps, unique_indices = np.unique(self.timestamps, return_index=True)
+
+        if len(unique_timestamps) != len(self.timestamps):
+            logger.warning(
+                "Timestamps contain duplicates. Only the first occurrence of each unique timestamp will be kept. "
+                "This may lead to loss of data if duplicates are present."
+            )
+            self.timestamps = unique_timestamps
+            self.positions.xyz = self.positions.xyz[unique_indices, :]
+
+            if self.rotations is not None:
+                self.rotations = Rotations.from_quat(self.rotations.as_quat()[unique_indices, :])
+
         if velocity_xyz is not None and len(velocity_xyz) == len(self.positions):
             self._velocity_xyz = velocity_xyz
         else:

{trajectopy-4.3.0 → trajectopy-4.3.1}/trajectopy/gui/managers/file_manager.py

@@ -61,9 +61,9 @@ class FileManager(QObject):
 
     @show_progress
     @Slot(FileRequest)
-    def handle_request(self, request: FileRequest) -> None:
+    def handle_request(self, request: FileRequest):
         """Logic for handling a request."""
-        self._request_executor.submit(generic_request_handler, self, request, True).result()
+        return self._request_executor.submit(generic_request_handler, self, request, True)
 
     def shutdown_executor(self) -> None:
         """Best-effort cleanup for the persistent Python worker thread."""

{trajectopy-4.3.0 → trajectopy-4.3.1}/trajectopy/gui/managers/trajectory_manager.py

@@ -196,8 +196,9 @@ class TrajectoryManager(QObject):
             None.
         """
         self.request = request
-        self._request_executor.submit(generic_request_handler, self, request, False).result()
-        self.update_view.emit()
+        future = self._request_executor.submit(generic_request_handler, self, request, False)
+        future.add_done_callback(lambda ft: self.update_view.emit())
+        return future
 
     def shutdown_executor(self) -> None:
         """Best-effort cleanup for the persistent Python worker thread."""

{trajectopy-4.3.0 → trajectopy-4.3.1}/trajectopy/gui/utils.py

@@ -61,8 +61,6 @@ def handle_drag_move(event: QtGui.QDragMoveEvent | None) -> None:
 def show_progress(func):
     """
     Decorator to show progress bar while executing a function
-
-    This should be used for functions that take a long time to execute.
     """
 
     @wraps(func)
@@ -71,12 +69,6 @@ def show_progress(func):
         progress_window = getattr(manager, "_progress_window", None)
 
         if progress_window is not None:
-            # Cross-thread: invoke the progress window slot directly via
-            # QMetaObject.invokeMethod with BlockingQueuedConnection.
-            # This guarantees the progress window is shown before work
-            # begins, even in PyInstaller-packaged executables where
-            # PySide6 signal dispatch across threads can be unreliable
-            # on the first call.
             QMetaObject.invokeMethod(
                 progress_window,
                 "handle_show_request",
@@ -86,15 +78,45 @@ def show_progress(func):
         manager.operation_started.emit()
 
         try:
-            return func(*args, **kwargs)
+            result = func(*args, **kwargs)
+            # Check if the result is a concurrent.futures.Future
+            import concurrent.futures
+
+            if isinstance(result, concurrent.futures.Future):
+
+                def on_done(future):
+                    # We must emit signals from the manager's thread
+                    # QMetaObject.invokeMethod handles this nicely if we call a slot,
+                    # but we can also just emit the signal directly since Qt queued connections will route it.
+                    manager.operation_finished.emit()
+                    if progress_window is not None:
+                        QMetaObject.invokeMethod(
+                            progress_window,
+                            "handle_close_request",
+                            Qt.ConnectionType.BlockingQueuedConnection,
+                        )
+                    # If there's an exception, we might want to log it
+                    try:
+                        future.result()
+                    except Exception as e:
+                        import logging
+
+                        logging.getLogger(__name__).exception("Error in background task: %s", e)
+
+                result.add_done_callback(on_done)
+                return result
+            else:
+                return result
         finally:
-            manager.operation_finished.emit()
-            if progress_window is not None:
-                QMetaObject.invokeMethod(
-                    progress_window,
-                    "handle_close_request",
-                    Qt.ConnectionType.BlockingQueuedConnection,
-                )
+            # Only emit finished immediately if it's NOT a Future
+            if not isinstance(result, concurrent.futures.Future):
+                manager.operation_finished.emit()
+                if progress_window is not None:
+                    QMetaObject.invokeMethod(
+                        progress_window,
+                        "handle_close_request",
+                        Qt.ConnectionType.BlockingQueuedConnection,
+                    )
 
     return wrapper
 

{trajectopy-4.3.0 → trajectopy-4.3.1}/trajectopy-docs/docs/settings.md

@@ -187,7 +187,7 @@ The ATE can be split into along-, horizontal-cross- and vertical-cross-track dir
 ### Mapbox Settings
 
 - `scatter_plot_on_map` (bool): Indicates whether the trajectory should be plotted onto a map. Default value is False.
-- `scatter_mapbox_style` (str): The style of the map. For some styles, a Mapbox token is required. Default value is "open-street-map".
+- `scatter_mapbox_style` (str): The style of the map. For some styles, a Mapbox token is required. Default value is "carto-positron".
 - `scatter_mapbox_zoom` (int): The zoom level of the map. Default value is 15.
 - `scatter_mapbox_token` (str): The Mapbox token. Default value is "".
 
@@ -239,7 +239,7 @@ To plot trajectories on a map, several requirements must be met:
 - The trajectory must have valid EPSG information.
 - The plotting backend must be set to `plotly`.
 - The `scatter_plot_on_map` option must be enabled.
-- For `scatter_mapbox_style` other than `open-street-map`, a Mapbox access token (`scatter_mapbox_token`) must be provided. The mapbox token can be obtained after free registration at [Mapbox](https://www.mapbox.com/).
+- For `scatter_mapbox_style` other than `carto-positron`, a Mapbox access token (`scatter_mapbox_token`) must be provided. The mapbox token can be obtained after free registration at [Mapbox](https://www.mapbox.com/).
 
 ## Matplotlib Plotting Settings (MPLPlotSettings)
 

trajectopy-4.3.0/trajectopy/__version__.py

@@ -1 +0,0 @@
-__version__ = "4.3.0"