Skip to content

Commit

Permalink
[python] try adding py312 to tests
Browse files Browse the repository at this point in the history
  • Loading branch information
hbaniecki committed Oct 2, 2024
1 parent 9269e3e commit fc081a4
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 7 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/Python-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:
python-version: ${{ matrix.python-version }}
- name: Install dependencies
run: |
python -m pip install --upgrade pip
python -m pip install --upgrade pip setuptools
pip install tox==3.28.0 tox-gh-actions==2.12.0
- name: Test with tox
run: tox
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import multiprocessing as mp
from distutils.version import LooseVersion

import numpy as np
import pandas as pd
Expand Down Expand Up @@ -74,11 +73,8 @@ def calculate_variable_profile(predict_function,

profiles = pd.concat(profile)
# convert the variable types
if LooseVersion(pd.__version__) >= LooseVersion('1.2.0'):
# convert_floating=False since pandas v1.2 seem to have issues
profiles.loc[:, list(variable_splits)] = profiles.loc[:, list(variable_splits)].convert_dtypes(convert_floating=False)
else:
profiles.loc[:, list(variable_splits)] = profiles.loc[:, list(variable_splits)].convert_dtypes()
# convert_floating=False since pandas v1.2 seem to have issues
profiles.loc[:, list(variable_splits)] = profiles.loc[:, list(variable_splits)].convert_dtypes(convert_floating=False)

return profiles

Expand Down
1 change: 1 addition & 0 deletions python/dalex/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,7 @@ def run_setup():
"Programming Language :: Python :: 3.9",
"Programming Language :: Python :: 3.10",
"Programming Language :: Python :: 3.11",
"Programming Language :: Python :: 3.12",
"License :: OSI Approved",
"License :: OSI Approved :: GNU General Public License v3 (GPLv3)",
"Operating System :: OS Independent",
Expand Down

0 comments on commit fc081a4

Please sign in to comment.