Update Python versions supported to 3.10 - 3.13.

Signed-off-by: Govind Kamat <[email protected]>

Author: gkamat

.ci/variables.json

@@ -1,9 +1,9 @@
 {
   "python_versions": {
-    "PY38": "3.8.12",
-    "PY39": "3.9.10",
     "PY310": "3.10.6",
     "PY311": "3.11.1",
-    "MIN_PY_VER": "3.8.12"
+    "PY312": "3.12.11",
+    "PY313": "3.13.7",
+    "MIN_PY_VER": "3.10.6"
   }
 }

.github/workflows/integ-test.yml

@@ -5,10 +5,10 @@ jobs:
     strategy:
       matrix:
         python-version:
-          - "3.8"
-          - "3.9"
           - "3.10"
           - "3.11"
+          - "3.12"
+          - "3.13"
     runs-on: ubuntu-latest
     steps:
       - name: Install bz2 development package

Makefile

@@ -19,7 +19,7 @@ SHELL = /bin/bash
 PYTHON = python3
 PIP = pip3
 VERSIONS = $(shell jq -r '.python_versions | .[]' .ci/variables.json | sed '$$d')
-VERSION38 = $(shell jq -r '.python_versions | .[]' .ci/variables.json | sed '$$d' | grep 3\.8)
+VERSION310 = $(shell jq -r '.python_versions | .[]' .ci/variables.json | sed '$$d' | grep 3\.10)
 PYENV_ERROR = "\033[0;31mIMPORTANT\033[0m: Please install pyenv and run \033[0;31meval \"\$$(pyenv init -)\"\033[0m.\n"
 
 all: develop
@@ -29,14 +29,14 @@ pyinst:
 	@for i in $(VERSIONS); do pyenv install --skip-existing $$i; done
 	pyenv local $(VERSIONS)
 
-pyinst38:
+pyinst310:
 	@which pyenv > /dev/null 2>&1 || { printf $(PYENV_ERROR); exit 1; }
-	pyenv install --skip-existing $(VERSION38)
-	pyenv local $(VERSION38)
+	pyenv install --skip-existing $(VERSION310)
+	pyenv local $(VERSION310)
 
 check-pip:
 	# Install pyenv if the Python environment is externally managed.
-	@if ! $(PIP) > /dev/null 2>&1 || ! $(PIP) install pip > /dev/null 2>&1; then make pyinst38; fi
+	@if ! $(PIP) > /dev/null 2>&1 || ! $(PIP) install pip > /dev/null 2>&1; then make pyinst310; fi
 
 check-java:
 	@if ! test "$(JAVA21_HOME)" || ! java --version > /dev/null 2>&1 || ! javadoc --help > /dev/null 2>&1; then \
@@ -52,7 +52,7 @@ install-deps: check-pip
 
 # pylint does not work with Python versions >3.8:
 #   Value 'Optional' is unsubscriptable (unsubscriptable-object)
-develop: pyinst38 install-deps
+develop: pyinst310 install-deps
 	PIP_ONLY_BINARY=h5py $(PIP) install -e .[develop]
 
 build: install-deps
@@ -88,7 +88,7 @@ it: pyinst check-java python-caches-clean tox-env-clean
 	@which tox || $(PIP) install tox
 	tox
 
-it38 it39 it310 it311: pyinst check-java python-caches-clean tox-env-clean
+it310 it311 it312 it313: pyinst check-java python-caches-clean tox-env-clean
 	@which tox || $(PIP) install tox
 	tox -e $(@:it%=py%)
 
@@ -106,4 +106,4 @@ release-checks:
 release: release-checks clean it
 	./release.sh $(release_version) $(next_version)
 
-.PHONY: install clean python-caches-clean tox-env-clean test it it38 benchmark coverage release release-checks pyinst
+.PHONY: install clean python-caches-clean tox-env-clean test it it310 benchmark coverage release release-checks pyinst

osbenchmark/__init__.py

@@ -80,7 +80,7 @@
 
 def check_python_version():
     if sys.version_info.major != 3 or sys.version_info.minor < 8:
-        raise RuntimeError("OSB requires at least Python 3.8 but you are using:\n\nPython %s" % str(sys.version))
+        raise RuntimeError("OSB requires at least Python 3.10 but you are using:\n\nPython %s" % str(sys.version))
 
 
 def doc_link(path=None):

osbenchmark/telemetry.py

@@ -367,7 +367,7 @@ def on_benchmark_start(self):
                                         self.indices_per_cluster[cluster_name] if self.indices_per_cluster else None)
             sampler = SamplerThread(recorder)
             self.samplers.append(sampler)
-            sampler.setDaemon(True)
+            sampler.daemon = True
             # we don't require starting recorders precisely at the same time
             sampler.start()
 
@@ -544,7 +544,7 @@ def on_benchmark_start(self):
                                              self.indices_per_cluster[cluster_name] if self.indices_per_cluster else "")
             sampler = SamplerThread(recorder)
             self.samplers.append(sampler)
-            sampler.setDaemon(True)
+            sampler.daemon = True
             # we don't require starting recorders precisely at the same time
             sampler.start()
 
@@ -761,7 +761,7 @@ def on_benchmark_start(self):
             recorder = NodeStatsRecorder(self.telemetry_params, cluster_name, self.clients[cluster_name], self.metrics_store)
             sampler = SamplerThread(recorder)
             self.samplers.append(sampler)
-            sampler.setDaemon(True)
+            sampler.daemon = True
             # we don't require starting recorders precisely at the same time
             sampler.start()
 
@@ -959,7 +959,7 @@ def on_benchmark_start(self):
                                                   cluster_name] if self.transforms_per_cluster else None)
             sampler = SamplerThread(recorder)
             self.samplers.append(sampler)
-            sampler.setDaemon(True)
+            sampler.daemon = True
             # we don't require starting recorders precisely at the same time
             sampler.start()
 
@@ -1170,7 +1170,7 @@ def on_benchmark_start(self):
                 self.indices_per_cluster[cluster_name] if self.indices_per_cluster else None)
             sampler = SamplerThread(recorder)
             self.samplers.append(sampler)
-            sampler.setDaemon(True)
+            sampler.daemon = True
             # we don't require starting recorders precisely at the same time
             sampler.start()
 
@@ -1866,7 +1866,7 @@ def on_benchmark_start(self):
                 self.indices_per_cluster[cluster_name] if self.indices_per_cluster else None)
             sampler = SamplerThread(recorder)
             self.samplers.append(sampler)
-            sampler.setDaemon(True)
+            sampler.daemon = True
             # we don't require starting recorders precisely at the same time
             sampler.start()
 

setup.py

@@ -44,7 +44,7 @@ def str_from_file(name):
 long_description = str_from_file("README.md")
 
 # tuples of (major, minor) of supported Python versions ordered from lowest to highest
-supported_python_versions = [(3, 8), (3, 9), (3, 10), (3, 11)]
+supported_python_versions = [(3, 10), (3, 11), (3, 13)]
 
 ################################################################################################
 #

tox.ini

@@ -7,7 +7,7 @@
 ###############################################################################
 [tox]
 envlist =
-    py38,py39,py310,py311
+    py310,py311,py312,py313
 platform =
     linux|darwin