Skip to content

Commit d8760d8

Browse files
Merge pull request #127 from aws/issue_106_0_23_1
Reverting numpy 1.21.2 to 1.19.2, updating boto3 to 1.24.17, fixing docker build and adding regression
2 parents f8015ce + 2268f2f commit d8760d8

File tree

4 files changed

+14
-5
lines changed

4 files changed

+14
-5
lines changed

docker/0.23-1/final/Dockerfile.cpu

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,8 @@ RUN python -m pip install -r /requirements.txt && \
99

1010
COPY dist/sagemaker_sklearn_container-2.0-py3-none-any.whl /sagemaker_sklearn_container-2.0-py3-none-any.whl
1111
# https://github.com/googleapis/google-cloud-python/issues/6647
12-
RUN rm -rf /miniconda3/lib/python3.7/site-packages/numpy-1.19.4.dist-info && \
12+
RUN conda install numpy --force-reinstall && \
13+
rm -rf /miniconda3/lib/python3.7/site-packages/numpy-1.19.4.dist-info && \
1314
pip install --no-cache /sagemaker_sklearn_container-2.0-py3-none-any.whl && \
1415
rm /sagemaker_sklearn_container-2.0-py3-none-any.whl
1516

requirements.txt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
boto3==1.16.4
2-
botocore==1.19.4
1+
boto3==1.24.17
2+
botocore==1.27.18
33
cryptography==35.0.0
44
Flask==1.1.1
55
itsdangerous==2.0.1
66
gunicorn==20.0.4
77
model-archiver==1.0.3
88
multi-model-server==1.1.1
9-
numpy==1.21.0
9+
numpy==1.19.2
1010
pandas==1.1.3
1111
psutil==5.7.2
1212
python-dateutil==2.8.1

test-requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
Flask
22
PyYAML
3-
boto3>=1.13.17
3+
boto3>=1.24.17
44
coverage
55
docker-compose
66
flake8

test/unit/test_modules.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,12 @@
1+
import pandas as pd
2+
3+
14
def test_pandas_version():
25
import pandas as pd
36
major, minor, patch = pd.__version__.split('.')
47
assert major == '1'
8+
9+
10+
def test_pyarrow_to_parquet_conversion_regression_issue_106():
11+
df = pd.DataFrame({'x': [1, 2]})
12+
df.to_parquet('test.parquet', engine='pyarrow')

0 commit comments

Comments
 (0)