Fix error with lowercase list for Python 3.8-3.9

.github/workflows/pip.yml

@@ -9,24 +9,20 @@ on:
 
 jobs:
   build:
-    name: Build with Pip
+    name: Install locally with pip
     runs-on: ${{ matrix.platform }}
     strategy:
       fail-fast: false
       matrix:
         platform: [macos-latest, ubuntu-latest, windows-latest]
-        python-version: ["3.12"]
-
+        python-version: ["3.8", "3.12"]
     steps:
     - uses: actions/checkout@v4
-
     - uses: actions/setup-python@v4
       with:
         python-version: ${{ matrix.python-version }}
-
     - run: pip install --upgrade pip wheel
     - name: Build and install
       run: pip install --verbose ".[test]"
-
     - name: Test
       run: pytest tests/

.github/workflows/wheels.yml

@@ -14,7 +14,6 @@ env:
   CIBW_TEST_COMMAND: pytest {project}/tests
   CIBW_TEST_EXTRAS: test
 
-
 jobs:
   build_sdist:
     name: Build SDist
@@ -23,18 +22,14 @@ jobs:
     - uses: actions/checkout@v4
       with:
         submodules: true
-
     - name: Build SDist
       run: pipx run build --sdist
-
     - name: Check metadata
       run: pipx run twine check dist/*
-
     - uses: actions/upload-artifact@v3
       with:
         path: dist/*.tar.gz
 
-
   build_wheels:
     name: Wheels on ${{ matrix.builds.os }}
     runs-on: ${{ matrix.builds.os }}
@@ -82,17 +77,14 @@ jobs:
     needs: [build_wheels, build_sdist]
     runs-on: ubuntu-latest
     if: github.event_name == 'release'
-
     steps:
     - uses: actions/setup-python@v4
       with:
         python-version: "3.x"
-
     - uses: actions/download-artifact@v3
       with:
         name: artifact
         path: dist
-
     - uses: pypa/gh-action-pypi-publish@release/v1
       with:
         user: __token__

CMakeLists.txt

@@ -1,6 +1,6 @@
 cmake_minimum_required(VERSION 3.15...3.28)
 
-project(fast_tsp VERSION "0.1.3")
+project(fast_tsp VERSION "0.1.4")
 
 if(SKBUILD)
   # Scikit-Build does not add your site-packages to the search path

conda.recipe/meta.yaml

@@ -1,6 +1,6 @@
 package:
   name: fast_tsp
-  version: 0.1.3
+  version: 0.1.4
 
 source:
   path: ..
@@ -24,7 +24,6 @@ requirements:
   run:
     - python
 
-
 test:
   imports:
     - fast_tsp

pyproject.toml

@@ -60,7 +60,7 @@ select = [
     "RUF",
 ]
 ignore-init-module-imports = true
-target-version = "py311"
+target-version = "py38"
 # show-source = true
 
 [tool.ruff.format]

src/fast_tsp/__init__.py

@@ -26,6 +26,8 @@
 from __future__ import annotations
 
 import warnings
+from typing import List
+from typing import Union
 
 import numpy as np
 
@@ -37,8 +39,8 @@
 from ._core import score_tour as __score_tour  # type: ignore
 from ._core import solve_tsp_exact as __solve_tsp_exact  # type: ignore
 
-Tour = list[int]
-DistMatrix = list[list[int]] | np.ndarray
+Tour = List[int]
+DistMatrix = Union[List[List[int]], np.ndarray]
 
 _UINT16_MAX = 2**16 - 1
 

tests/basic_test.py

@@ -1,9 +1,33 @@
 from __future__ import annotations
 
+import re
+from pathlib import Path
+
 import fast_tsp
 import pytest
 
 
+def test_version_is_consistent():
+    base_dir = Path(__file__).resolve().parent.parent
+
+    cmake_version = ''
+    cmake_file = base_dir / 'CMakeLists.txt'
+    for line in cmake_file.read_text(encoding='utf-8').splitlines():
+        line_match = re.match(r'project\(fast_tsp VERSION "([\d\.]+)"\)', line)
+        if line_match:
+            cmake_version = line_match.group(1)
+            break
+
+    conda_version = ''
+    conda_recipe = base_dir / 'conda.recipe' / 'meta.yaml'
+    for line in conda_recipe.read_text(encoding='utf-8').splitlines():
+        line_match = re.match(r' *version: ([\d\.]+)', line)
+        if line_match:
+            conda_version = line_match.group(1)
+            break
+    assert cmake_version and cmake_version == conda_version
+
+
 def test_errors_with_wrong_signature():
     with pytest.raises(ValueError, match='Distance matrix must be 2D'):
         fast_tsp.find_tour([0])  # type: ignore