Skip to content

Commit

Permalink
chore: support python 3.12 (#76)
Browse files Browse the repository at this point in the history
* support python 3.12

* typo

* remove extra space

* try to skip root on python 3.12

* maybe this will work
  • Loading branch information
ikrommyd authored Apr 13, 2024
1 parent ed74592 commit 4b2333d
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 12 deletions.
14 changes: 6 additions & 8 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ jobs:
max-parallel: 4
fail-fast: False
matrix:
python-version: [ "3.8", "3.9", "3.10", "3.11" ]
python-version: [ "3.9", "3.10", "3.11", "3.12" ]
name: JIT compiled tests for Python ${{ matrix.python-version }}
steps:
- uses: actions/checkout@v4
Expand All @@ -42,15 +42,14 @@ jobs:
with:
environment-name: test-env
create-args: >-
python=${{ matrix.python-version }}
pip
root
python=${{ matrix.python-version }} pip ${{ matrix.python-version != '3.12' && 'root' || '' }}
- name: Install Python dependencies
run: |
which python
python -V
python -m pip install uv
python -m uv pip install -e ".[dev]"
python -m uv pip install git+https://github.com/HDembinski/numba-stats.git
- name: Test with pytest
run: |
coverage run --source=. --omit=".tox/*" --branch -m pytest .
Expand All @@ -63,7 +62,7 @@ jobs:
max-parallel: 4
fail-fast: False
matrix:
python-version: [ "3.8", "3.9", "3.10", "3.11" ]
python-version: [ "3.9", "3.10", "3.11", "3.12" ]
name: Eager mode tests for Python ${{ matrix.python-version }}
steps:
- uses: actions/checkout@v4
Expand All @@ -72,15 +71,14 @@ jobs:
with:
environment-name: test-env
create-args: >-
python=${{ matrix.python-version }}
pip
root
python=${{ matrix.python-version }} pip ${{ matrix.python-version != '3.12' && 'root' || '' }}
- name: Install Python dependencies
run: |
which python
python -V
python -m pip install uv
python -m uv pip install -e ".[dev]"
python -m uv pip install git+https://github.com/HDembinski/numba-stats.git
- name: Test with pytest
run: |
ZFIT_DO_JIT=0
Expand Down
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ repos:
rev: v2.5.0
hooks:
- id: setup-cfg-fmt
args: [ --max-py-version=3.10, --include-version-classifiers ]
args: [ --max-py-version=3.12, --include-version-classifiers ]


# Notebook formatting
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ Major Features and Improvements
- added ErfExp PDF
- added Novosibirsk PDF
- added Tsallis PDF
- upgrade to zfit>=0.20, support Python 3.9-3.12

Breaking changes
------------------
Expand Down
2 changes: 1 addition & 1 deletion requirements_dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ coverage>=4.5.1
flake8>=3.5.0
jupyter-sphinx
myst-nb
numba-stats @ git+https://github.com/HDembinski/numba-stats.git # CMSShape not yet released (expected 1.8.0)
numba-stats # hack for pypi release, doesn't allow git repos @ git+https://github.com/HDembinski/numba-stats.git # CMSShape not yet released (expected 1.8.0)
pip>=9.0.1
pre-commit
pydata-sphinx-theme>=0.9 # new dark theme configuration
Expand Down
5 changes: 3 additions & 2 deletions setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -19,15 +19,16 @@ classifiers =
Operating System :: Unix
Programming Language :: Python :: 3
Programming Language :: Python :: 3 :: Only
Programming Language :: Python :: 3.8
Programming Language :: Python :: 3.9
Programming Language :: Python :: 3.10
Programming Language :: Python :: 3.11
Programming Language :: Python :: 3.12
Programming Language :: Python :: Implementation :: CPython
Topic :: Scientific/Engineering :: Physics
keywords = TensorFlow, model, fitting, scalable, HEP, physics

[options]
python_requires = >=3.8
python_requires = >=3.9

[bdist_wheel]
universal = 1
Expand Down

0 comments on commit 4b2333d

Please sign in to comment.