From 04ce097ec918e3d23329795ceb344f36955f7926 Mon Sep 17 00:00:00 2001 From: jiamingy Date: Thu, 31 Mar 2022 01:58:36 +0800 Subject: [PATCH 1/5] Drop support for deprecated CUDA architecture. I can't upload the pip package due to maximum size reached. ``` HTTPError: 400 Bad Request from https://upload.pypi.org/legacy/ File too large. Limit for project 'xgboost' is 200 MB. See https://pypi.org/help/#file-size-limit for more information. ``` --- cmake/Utils.cmake | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cmake/Utils.cmake b/cmake/Utils.cmake index 073f36b1f828..963c494ccf26 100644 --- a/cmake/Utils.cmake +++ b/cmake/Utils.cmake @@ -91,9 +91,9 @@ function(format_gencode_flags flags out) # Set up architecture flags if(NOT flags) if (CUDA_VERSION VERSION_GREATER_EQUAL "11.1") - set(flags "50;52;60;61;70;75;80;86") + set(flags "52;60;61;70;75;80;86") elseif (CUDA_VERSION VERSION_GREATER_EQUAL "11.0") - set(flags "35;50;52;60;61;70;75;80") + set(flags "52;60;61;70;75;80") elseif(CUDA_VERSION VERSION_GREATER_EQUAL "10.0") set(flags "35;50;52;60;61;70;75") elseif(CUDA_VERSION VERSION_GREATER_EQUAL "9.0") From 07f334d401345320fc7bfab92411d8eab67ea671 Mon Sep 17 00:00:00 2001 From: jiamingy Date: Thu, 31 Mar 2022 03:27:48 +0800 Subject: [PATCH 2/5] Check file size at release branch. --- tests/ci_build/rename_whl.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/tests/ci_build/rename_whl.py b/tests/ci_build/rename_whl.py index e615d2e2e5c1..1b642b5c9524 100644 --- a/tests/ci_build/rename_whl.py +++ b/tests/ci_build/rename_whl.py @@ -2,6 +2,7 @@ import os from contextlib import contextmanager + @contextmanager def cd(path): path = os.path.normpath(path) @@ -13,10 +14,12 @@ def cd(path): finally: os.chdir(cwd) + if len(sys.argv) != 4: print('Usage: {} [wheel to rename] [commit id] [platform tag]'.format(sys.argv[0])) sys.exit(1) + whl_path = sys.argv[1] commit_id = sys.argv[2] platform_tag = sys.argv[3] @@ -36,3 +39,7 @@ def cd(path): if os.path.isfile(new_name): os.remove(new_name) os.rename(basename, new_name) + + filesize = os.path.getsize(new_name) / 1024 / 1024 # MB + msg = f"Limit of wheel size set by PyPI is exceeded. {new_name}: {filesize}" + assert filesize <= 100, msg From 3e54d793ee9b4a7e0cdea0d7011b50ba8c91ba17 Mon Sep 17 00:00:00 2001 From: Philip Hyunsu Cho Date: Wed, 30 Mar 2022 13:08:16 -0700 Subject: [PATCH 3/5] Use 200 MB limit --- tests/ci_build/rename_whl.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/ci_build/rename_whl.py b/tests/ci_build/rename_whl.py index 1b642b5c9524..f4f8bd41f02a 100644 --- a/tests/ci_build/rename_whl.py +++ b/tests/ci_build/rename_whl.py @@ -42,4 +42,4 @@ def cd(path): filesize = os.path.getsize(new_name) / 1024 / 1024 # MB msg = f"Limit of wheel size set by PyPI is exceeded. {new_name}: {filesize}" - assert filesize <= 100, msg + assert filesize <= 200, msg From 55d49e944d2d0fc8746f8f7879d2595851691479 Mon Sep 17 00:00:00 2001 From: Hyunsu Cho Date: Wed, 30 Mar 2022 13:24:58 -0700 Subject: [PATCH 4/5] Build all archs (temporarily) --- Jenkinsfile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Jenkinsfile b/Jenkinsfile index 7034329215c8..9622371d424e 100644 --- a/Jenkinsfile +++ b/Jenkinsfile @@ -211,9 +211,9 @@ def BuildCUDA(args) { def docker_binary = "docker" def docker_args = "--build-arg CUDA_VERSION_ARG=${args.cuda_version}" def arch_flag = "" - if (env.BRANCH_NAME != 'master' && !(env.BRANCH_NAME.startsWith('release'))) { - arch_flag = "-DGPU_COMPUTE_VER=75" - } + //if (env.BRANCH_NAME != 'master' && !(env.BRANCH_NAME.startsWith('release'))) { + // arch_flag = "-DGPU_COMPUTE_VER=75" + //} def wheel_tag = "manylinux2014_x86_64" sh """ ${dockerRun} ${container_type} ${docker_binary} ${docker_args} tests/ci_build/build_via_cmake.sh -DUSE_CUDA=ON -DUSE_NCCL=ON -DOPEN_MP:BOOL=ON -DHIDE_CXX_SYMBOLS=ON ${arch_flag} From 36c11293fa1fe1697384ede781f3234d86584a93 Mon Sep 17 00:00:00 2001 From: jiamingy Date: Thu, 31 Mar 2022 04:45:13 +0800 Subject: [PATCH 5/5] Revert "Build all archs (temporarily)" This reverts commit 55d49e944d2d0fc8746f8f7879d2595851691479. --- Jenkinsfile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Jenkinsfile b/Jenkinsfile index 9622371d424e..7034329215c8 100644 --- a/Jenkinsfile +++ b/Jenkinsfile @@ -211,9 +211,9 @@ def BuildCUDA(args) { def docker_binary = "docker" def docker_args = "--build-arg CUDA_VERSION_ARG=${args.cuda_version}" def arch_flag = "" - //if (env.BRANCH_NAME != 'master' && !(env.BRANCH_NAME.startsWith('release'))) { - // arch_flag = "-DGPU_COMPUTE_VER=75" - //} + if (env.BRANCH_NAME != 'master' && !(env.BRANCH_NAME.startsWith('release'))) { + arch_flag = "-DGPU_COMPUTE_VER=75" + } def wheel_tag = "manylinux2014_x86_64" sh """ ${dockerRun} ${container_type} ${docker_binary} ${docker_args} tests/ci_build/build_via_cmake.sh -DUSE_CUDA=ON -DUSE_NCCL=ON -DOPEN_MP:BOOL=ON -DHIDE_CXX_SYMBOLS=ON ${arch_flag}