update to min py3.9 in build

Author: apoorvkh

.github/workflows/main.yml

@@ -67,15 +67,15 @@ jobs:
     strategy: 
       fail-fast: false
       matrix:
-        python: ["3.8", "3.9", "3.10", "3.11", "3.12"] 
+        python: ["3.9", "3.10", "3.11", "3.12"] 
         pytorch: ${{fromJson(needs.get-pytorch-versions.outputs.versions)}}
 
     steps:
       - uses: actions/checkout@v4
 
       - name: Inject (python / pytorch) test deps into pixi.toml / pyproject.toml
         run: |
-          sed -i 's/python = "3.8.1"/python = "${{ matrix.python }}.*"/' pixi.toml
+          sed -i 's/python = "3.9"/python = "${{ matrix.python }}.*"/' pixi.toml
           sed -i 's/torch>=2.0.0/torch~=${{ matrix.pytorch }}/' pyproject.toml
 
       - uses: prefix-dev/[email protected]

README.md

@@ -16,7 +16,7 @@ By [Apoorv Khandelwal](http://apoorvkh.com) and [Peter Curtin](https://github.co
 pip install torchrunx
 ```
 
-Requires: Linux, Python >= 3.8.1, PyTorch >= 2.0
+Requires: Linux, Python >= 3.9, PyTorch >= 2.0
 
 Shared filesystem & SSH access if using multiple machines
 

docs/.readthedocs.yaml

@@ -3,7 +3,7 @@ version: 2
 build:
   os: ubuntu-24.04
   tools:
-    python: "3.8"
+    python: "3.9"
 
 sphinx:
   configuration: docs/source/conf.py

src/torchrunx/launcher.py

@@ -136,7 +136,7 @@ def execute_command(
         is_localhost = len(set(_host_addrs) & set(_localhost_addrs)) > 0
 
     if is_localhost:
-        # S602: subprocess.Popen is called with shell=True (https://docs.python.org/3.8/library/subprocess.html#security-considerations)
+        # S602: subprocess.Popen is called with shell=True (https://docs.python.org/3.9/library/subprocess.html#security-considerations)
         # Made sure to shlex.quote arguments in build_command to prevent shell injection
         subprocess.Popen(command, shell=True, stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)  # noqa: S602
     else: