Skip to content

Commit 6dcd442

Browse files
malav-shastriMalav Shastri
andauthored
Upgrade XgBoost to 1.7.x (#362)
* Upgrade XgBoost version to 1.7.3 * Restructure checkpointing.py and fix Unit tests * Resolve Flake8 style errors * Address Comments and improvements * Add comments and improvements * Upgrade xgboost to 1.7.4 * remove grow_local_histmaker and Single precision histogram --------- Co-authored-by: Malav Shastri <[email protected]>
1 parent 3aa31e9 commit 6dcd442

File tree

14 files changed

+167
-137
lines changed

14 files changed

+167
-137
lines changed

README.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -253,4 +253,4 @@ SageMaker XGboost Framework Container is licensed under the Apache 2.0 License.
253253
.com, Inc. or its affiliates. All Rights Reserved. The license is available at:
254254
http://aws.amazon.com/apache2.0/
255255

256-
.. |XGBoostLatestVersion| replace:: 1.5-1
256+
.. |XGBoostLatestVersion| replace:: 1.7-1

docker/1.5-1/base/Dockerfile.cpu renamed to docker/1.7-1/base/Dockerfile.cpu

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ ARG CONDA_PKG_VERSION=4.10.1
1111
ARG PYTHON_VERSION=3.8.13
1212
ARG PYARROW_VERSION=1.0
1313
ARG MLIO_VERSION=0.7.0
14-
ARG XGBOOST_VERSION=1.5.2
14+
ARG XGBOOST_VERSION=1.7.4
1515

1616
ENV DEBIAN_FRONTEND=noninteractive
1717
ENV LANG=C.UTF-8
File renamed without changes.

docker/1.5-1/final/Dockerfile.cpu renamed to docker/1.7-1/final/Dockerfile.cpu

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
ARG SAGEMAKER_XGBOOST_VERSION=1.5-1
1+
ARG SAGEMAKER_XGBOOST_VERSION=1.7-1
22
ARG PYTHON_VERSION=3.8
33

44
FROM xgboost-container-base:${SAGEMAKER_XGBOOST_VERSION}-cpu-py3
File renamed without changes.

requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ PyYAML==5.4.1
33
Pillow==9.1.1
44
boto3==1.17.52
55
botocore==1.20.52
6-
cryptography==35.0.0
6+
cryptography==39.0.1
77
dask==2022.11.1
88
dask-cuda==22.12.0
99
gunicorn==19.10.0

src/sagemaker_xgboost_container/algorithm_mode/hyperparameter_validation.py

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ def updater_validator(value, dependencies):
2929
"grow_colmaker",
3030
"distcol",
3131
"grow_histmaker",
32-
"grow_local_histmaker",
3332
"grow_skmaker",
3433
"sync",
3534
"refresh",
@@ -40,7 +39,6 @@ def updater_validator(value, dependencies):
4039
"grow_colmaker",
4140
"distcol",
4241
"grow_histmaker",
43-
"grow_local_histmaker",
4442
"grow_colmaker",
4543
"grow_quantile_histmaker",
4644
]
@@ -62,7 +60,7 @@ def updater_validator(value, dependencies):
6260
if not all(x in valid_tree_plugins for x in value):
6361
raise exc.UserError(
6462
"Tree updater should be selected from these options: 'grow_colmaker', 'distcol', 'grow_histmaker', "
65-
"'grow_local_histmaker', 'grow_skmaker', 'grow_quantile_histmaker', 'sync', 'refresh', 'prune', "
63+
"'grow_skmaker', 'grow_quantile_histmaker', 'sync', 'refresh', 'prune', "
6664
"'shortgun', 'coord_descent'."
6765
)
6866
# validate only one tree updater is selected
@@ -74,7 +72,7 @@ def updater_validator(value, dependencies):
7472
raise exc.UserError(
7573
"Only one tree grow plugin can be selected. Choose one from the"
7674
"following: 'grow_colmaker', 'distcol', 'grow_histmaker', "
77-
"'grow_local_histmaker', 'grow_skmaker'"
75+
"'grow_skmaker'"
7876
)
7977

8078
@hpv.range_validator(["auto", "cpu_predictor", "gpu_predictor"])
@@ -239,15 +237,13 @@ def interaction_constraints_validator(value, dependencies):
239237
"grow_colmaker",
240238
"distcol",
241239
"grow_histmaker",
242-
"grow_local_histmaker",
243240
"grow_skmaker",
244241
"sync",
245242
"refresh",
246243
"prune",
247244
"grow_colmaker",
248245
"distcol",
249246
"grow_histmaker",
250-
"grow_local_histmaker",
251247
"grow_colmaker",
252248
"shotgun",
253249
"coord_descent",
@@ -334,7 +330,6 @@ def interaction_constraints_validator(value, dependencies):
334330
hpv.ContinuousHyperparameter(
335331
name="aft_loss_distribution_scale", range=hpv.Interval(min_closed=0), required=False
336332
),
337-
hpv.CategoricalHyperparameter(name="single_precision_histogram", range=["true", "false"], required=False),
338333
hpv.CategoricalHyperparameter(name="deterministic_histogram", range=["true", "false"], required=False),
339334
hpv.CategoricalHyperparameter(name="sampling_method", range=["uniform", "gradient_based"], required=False),
340335
hpv.IntegerHyperparameter(name="prob_buffer_row", range=hpv.Interval(min_open=1.0), required=False),

0 commit comments

Comments
 (0)