Skip to content

Commit

Permalink
Upgrade to nvcomp 4.0.1 (#16076)
Browse files Browse the repository at this point in the history
This PR bumps nvcomp to 4.0.1.

Depends on:
- conda-forge/nvcomp-feedstock#15
- rapidsai/rapids-cmake#633
- rapidsai/kvikio#449

Authors:
  - Vukasin Milovanovic (https://github.com/vuule)
  - Robert Maynard (https://github.com/robertmaynard)
  - Peixin (https://github.com/pxLi)
  - Bradley Dice (https://github.com/bdice)
  - Nghia Truong (https://github.com/ttnghia)

Approvers:
  - Nghia Truong (https://github.com/ttnghia)
  - Bradley Dice (https://github.com/bdice)
  - Robert Maynard (https://github.com/robertmaynard)

URL: #16076
  • Loading branch information
vuule authored Sep 5, 2024
1 parent 949f171 commit 0cc059f
Show file tree
Hide file tree
Showing 10 changed files with 8 additions and 19 deletions.
2 changes: 0 additions & 2 deletions ci/build_wheel_cudf.sh
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,6 @@ export PIP_CONSTRAINT="/tmp/constraints.txt"
python -m auditwheel repair \
--exclude libcudf.so \
--exclude libnvcomp.so \
--exclude libnvcomp_bitcomp.so \
--exclude libnvcomp_gdeflate.so \
-w ${package_dir}/final_dist \
${package_dir}/dist/*

Expand Down
2 changes: 0 additions & 2 deletions ci/build_wheel_pylibcudf.sh
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,6 @@ export PIP_CONSTRAINT="/tmp/constraints.txt"
python -m auditwheel repair \
--exclude libcudf.so \
--exclude libnvcomp.so \
--exclude libnvcomp_bitcomp.so \
--exclude libnvcomp_gdeflate.so \
-w ${package_dir}/final_dist \
${package_dir}/dist/*

Expand Down
2 changes: 1 addition & 1 deletion conda/environments/all_cuda-118_arch-x86_64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ dependencies:
- numpy>=1.23,<3.0a0
- numpydoc
- nvcc_linux-64=11.8
- nvcomp==3.0.6
- nvcomp==4.0.1
- nvtx>=0.2.1
- openpyxl
- packaging
Expand Down
2 changes: 1 addition & 1 deletion conda/environments/all_cuda-125_arch-x86_64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ dependencies:
- numba>=0.57
- numpy>=1.23,<3.0a0
- numpydoc
- nvcomp==3.0.6
- nvcomp==4.0.1
- nvtx>=0.2.1
- openpyxl
- packaging
Expand Down
2 changes: 1 addition & 1 deletion conda/recipes/libcudf/conda_build_config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ spdlog_version:
- ">=1.12.0,<1.13"

nvcomp_version:
- "=3.0.6"
- "=4.0.1"

zlib_version:
- ">=1.2.13"
Expand Down
2 changes: 1 addition & 1 deletion dependencies.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -354,7 +354,7 @@ dependencies:
- flatbuffers==24.3.25
- librdkafka>=1.9.0,<1.10.0a0
# Align nvcomp version with rapids-cmake
- nvcomp==3.0.6
- nvcomp==4.0.1
- spdlog>=1.12.0,<1.13
rapids_build_skbuild:
common:
Expand Down
4 changes: 1 addition & 3 deletions java/pom.xml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?xml version="1.0" encoding="UTF-8"?>
<!--
Copyright (c) 2019-2023, NVIDIA CORPORATION.
Copyright (c) 2019-2024, NVIDIA CORPORATION.
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -590,8 +590,6 @@
<include>libcudfjni.so</include>
<include>libcufilejni.so</include>
<include>libnvcomp.so</include>
<include>libnvcomp_gdeflate.so</include>
<include>libnvcomp_bitcomp.so</include>
</includes>
</resource>
<resource>
Expand Down
3 changes: 0 additions & 3 deletions java/src/main/java/ai/rapids/cudf/NativeDepsLoader.java
Original file line number Diff line number Diff line change
Expand Up @@ -54,9 +54,6 @@ public class NativeDepsLoader {
* subsequent stages are loaded.
*/
private static final String[][] loadOrder = new String[][]{
new String[]{
"nvcomp_bitcomp", "nvcomp_gdeflate"
},
new String[]{
"nvcomp"
},
Expand Down
5 changes: 2 additions & 3 deletions java/src/main/native/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -267,9 +267,8 @@ if(TARGET nvcomp::nvcomp)
add_custom_command(
TARGET cudfjni
PRE_LINK
COMMAND
${CMAKE_COMMAND} -E copy $<TARGET_FILE:nvcomp::nvcomp> $<TARGET_FILE:nvcomp::nvcomp_gdeflate>
$<TARGET_FILE:nvcomp::nvcomp_bitcomp> "${PROJECT_BINARY_DIR}"
COMMAND ${CMAKE_COMMAND} -E copy $<TARGET_FILE:nvcomp::nvcomp>
"${PROJECT_BINARY_DIR}/libnvcomp.so"
COMMENT "Copying nvcomp libraries to ${PROJECT_BINARY_DIR}"
)
endif()
3 changes: 1 addition & 2 deletions python/libcudf/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,5 @@ add_subdirectory(../../cpp cudf-cpp)
# Ensure other libraries needed by libcudf.so get installed alongside it.
include(cmake/Modules/WheelHelpers.cmake)
install_aliased_imported_targets(
TARGETS cudf nvcomp::nvcomp nvcomp::nvcomp_gdeflate nvcomp::nvcomp_bitcomp DESTINATION
${CMAKE_LIBRARY_OUTPUT_DIRECTORY}
TARGETS cudf nvcomp::nvcomp DESTINATION ${CMAKE_LIBRARY_OUTPUT_DIRECTORY}
)

0 comments on commit 0cc059f

Please sign in to comment.