tlpytools 0.2.1__tar.gz → 0.2.3__tar.gz

{tlpytools-0.2.1 → tlpytools-0.2.3}/.github/copilot-instructions.md

@@ -98,8 +98,8 @@ Interpretation for agent PRs:
 
 Observed result:
 - Build succeeds and produces:
-  - `dist/tlpytools-0.2.1.tar.gz`
-  - `dist/tlpytools-0.2.1-py3-none-any.whl`
+  - `dist/tlpytools-0.2.3.tar.gz`
+  - `dist/tlpytools-0.2.3-py3-none-any.whl`
 
 ### 5) Run
 - ORCA CLI (module form):

{tlpytools-0.2.1 → tlpytools-0.2.3}/.github/workflows/deploy.yml

@@ -4,7 +4,7 @@ on:
   workflow_dispatch:
     inputs:
       version:
-        description: 'Version number to release (e.g., 0.2.1)'
+        description: 'Version number to release (e.g., 0.2.3)'
         required: true
         type: string
       prerelease:

{tlpytools-0.2.1 → tlpytools-0.2.3}/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 All notable changes to TLPyTools will be documented in this file.
 
+## [0.2.3]
+
+### Fixed
+* Fix python cmd syntax issue introduced in 0.2.2
+
+## [0.2.2]
+
+### Fixed
+* Fix python path leaks by cleaning up PYTHONPATH and PYTHONHOME during sub component model runs 
+* Clear stale errors when resuming or starting new runs on old scenario folders
+
 ## [0.2.1]
 
 ### Changed

{tlpytools-0.2.1 → tlpytools-0.2.3}/DEVELOPER_GUIDE.md

@@ -200,7 +200,7 @@ make check-all
 4. Click **"Run workflow"** button
 5. Fill in the required information:
    - **Branch**: Select `main` (or your release branch)
-   - **Version number**: Enter the new version (e.g., `0.2.1`)
+   - **Version number**: Enter the new version (e.g., `0.2.3`)
    - **Is this a pre-release?**: Check if this is a pre-release version
 
 #### 3. Monitor the Release
@@ -219,7 +219,7 @@ After the workflow completes:
 
 ```bash
 # Verify the package is available on PyPI
-pip install tlpytools==0.2.1  # Replace with your version
+pip install tlpytools==0.2.3  # Replace with your version
 
 # Test installation in a fresh environment
 python -c "import tlpytools; print(tlpytools.__version__)"
@@ -235,8 +235,8 @@ TLPyTools follows semantic versioning (SemVer):
 
 Examples:
 - `0.1.7` → `0.1.8` (patch release)
-- `0.1.8` → `0.2.1` (minor release)
-- `0.2.1` → `1.0.0` (major release)
+- `0.1.8` → `0.2.3` (minor release)
+- `0.2.3` → `1.0.0` (major release)
 
 ### Pre-releases
 

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tlpytools
-Version: 0.2.1
+Version: 0.2.3
 Summary: TransLink's Python Tools - A comprehensive toolkit for transportation modeling and forecasting
 Project-URL: Homepage, https://github.com/TransLinkForecasting/tlpytools
 Project-URL: Bug Tracker, https://github.com/TransLinkForecasting/tlpytools/issues

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

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "tlpytools"
-version = "0.2.1"
+version = "0.2.3"
 description = "TransLink's Python Tools - A comprehensive toolkit for transportation modeling and forecasting"
 readme = "README.md"
 requires-python = ">=3.10.0, <3.13"

{tlpytools-0.2.1 → tlpytools-0.2.3}/src/tlpytools/__init__.py

@@ -2,7 +2,7 @@
 TLPyTools - A set of tools for building models at the TransLink Forecasting Team
 """
 
-__version__ = "0.2.1"
+__version__ = "0.2.3"
 
 # Core modules that should always be available (minimal dependencies)
 from . import log

{tlpytools-0.2.1 → tlpytools-0.2.3}/src/tlpytools/orca/orchestrator.py

@@ -3737,16 +3737,20 @@ class OrcaOrchestrator:
 
                 # Create a platform-specific command to set the PYTHONPATH
                 # Use step_directory instead of os.getcwd() for consistency
+                # PYTHONHOME and PYTHONPATH are cleared first to prevent the parent
+                # Python process (e.g. Jupyter/tlpytools env) from leaking its
+                # library paths into the component subprocess
                 if os.name == "nt":  # Windows
-                    # On Windows, use SET command
+                    # Use quoted SET form to clear PYTHONHOME without risk of
+                    # capturing trailing whitespace (bare "set VAR= " would set
+                    # VAR to a space, breaking the Python stdlib path lookup).
                     pythonpath_cmd = (
-                        f"set PYTHONPATH={ORCA_DIR};{step_directory};%PYTHONPATH% && "
+                        f'set "PYTHONHOME=" && '
+                        f'set "PYTHONPATH={ORCA_DIR};{step_directory}" && '
                     )
                 else:  # Unix/Linux/Mac
                     # On Unix-like systems, use export command
-                    pythonpath_cmd = (
-                        f"export PYTHONPATH={ORCA_DIR}:{step_directory}:$PYTHONPATH && "
-                    )
+                    pythonpath_cmd = f"unset PYTHONHOME && export PYTHONPATH={ORCA_DIR}:{step_directory} && "
 
                 # Modify the command to ensure environment can be found
                 modified_command = f"{pythonpath_cmd}{command}"
@@ -6269,8 +6273,8 @@ class OrcaOrchestrator:
         # Reload configuration to pick up any YAML changes
         self.reload_configuration()
 
-        # Update status
-        self.state.update(status="running")
+        # Update status and clear any stale error from a previous failed run
+        self.state.update(status="running", error=None)
 
         # Add safety counter to prevent infinite loops
         max_steps = self.state.get("total_iterations", 1) * len(
@@ -6334,8 +6338,8 @@ class OrcaOrchestrator:
                 self.logger.error("Error executing step: %s", step_name)
                 return False
 
-        # Update state to reflect completion
-        self.state.update(status="completed")
+        # Update state to reflect completion and clear any previously recorded error
+        self.state.update(status="completed", error=None)
         self.logger.info("Model run completed successfully")
         return True
 
@@ -6356,8 +6360,8 @@ class OrcaOrchestrator:
         # Reload configuration to pick up any YAML changes
         self.reload_configuration()
 
-        # Update status
-        self.state.update(status="running")
+        # Update status and clear any stale error from a previous failed run
+        self.state.update(status="running", error=None)
 
         # Add safety counter to prevent infinite loops
         max_steps = self.state.get("total_iterations", 1) * len(
@@ -6437,8 +6441,8 @@ class OrcaOrchestrator:
 
                 return False
 
-        # Update state to reflect completion
-        self.state.update(status="completed")
+        # Update state to reflect completion and clear any previously recorded error
+        self.state.update(status="completed", error=None)
 
         # Final sync to cloud after completion
         self.logger.info("Final state sync to cloud after model completion")
@@ -6495,12 +6499,12 @@ class OrcaOrchestrator:
                 current_iter,
                 max_iteration,
             )
-            self.state.update(status="completed")
+            self.state.update(status="completed", error=None)
             return True
 
-        # Update status to running if not already
+        # Update status to running if not already, clearing any stale error
         if self.state.get("status") != "running":
-            self.state.update(status="running")
+            self.state.update(status="running", error=None)
 
         # Get the next step to execute
         step_name = self.state.get_next_step()
@@ -6517,7 +6521,7 @@ class OrcaOrchestrator:
             # Check if we've exceeded the total iterations
             if self.state.is_all_iterations_complete():
                 self.logger.info("All iterations completed")
-                self.state.update(status="completed")
+                self.state.update(status="completed", error=None)
                 return True
 
             # Get the first step of the new iteration
@@ -6552,7 +6556,7 @@ class OrcaOrchestrator:
 
             # Check if we just completed the last iteration
             if self.state.is_all_iterations_complete():
-                self.state.update(status="completed")
+                self.state.update(status="completed", error=None)
                 self.logger.info("Model run completed successfully")
 
             self.logger.info("Step '%s' completed successfully", step_name)
@@ -6597,12 +6601,12 @@ class OrcaOrchestrator:
                 current_iter,
                 max_iteration,
             )
-            self.state.update(status="completed")
+            self.state.update(status="completed", error=None)
             return True
 
-        # Update status to running if not already
+        # Update status to running if not already, clearing any stale error
         if self.state.get("status") != "running":
-            self.state.update(status="running")
+            self.state.update(status="running", error=None)
 
         # Remember the starting iteration
         starting_iter = self.state.get_display_iteration()
@@ -6624,7 +6628,7 @@ class OrcaOrchestrator:
 
                 # Check if we've completed all iterations
                 if self.state.is_all_iterations_complete():
-                    self.state.update(status="completed")
+                    self.state.update(status="completed", error=None)
                     self.logger.info("All iterations completed")
 
                 return True