Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate a few CUB macros #3456

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

bernhardmgruber
Copy link
Contributor

@bernhardmgruber bernhardmgruber commented Jan 21, 2025

Deprecate CUB_MAX, CUB_MIN, CUB_QUOTIENT_FLOOR, CUB_QUOTIENT_CEILING, CUB_ROUND_UP_NEAREST, and CUB_ROUND_DOWN_NEAREST. No replacements are being made yet, because this PR should be backported to 2.8.x and we may need C++17 for some of the replacements. So let's not backport the replacements.

Copy link
Contributor

🟩 CI finished in 1h 54m: Pass: 100%/78 | Total: 1d 17h | Avg: 31m 37s | Max: 1h 18m | Hits: 138%/12720
  • 🟩 cub: Pass: 100%/38 | Total: 1d 01h | Avg: 40m 49s | Max: 1h 18m | Hits: 43%/3540

    🟩 cpu
      🟩 amd64              Pass: 100%/36  | Total:  1d 00h | Avg: 40m 36s | Max:  1h 18m | Hits:  43%/3540  
      🟩 arm64              Pass: 100%/2   | Total:  1h 29m | Avg: 44m 52s | Max: 45m 21s
    🟩 ctk
      🟩 12.0               Pass: 100%/5   | Total:  3h 45m | Avg: 45m 08s | Max:  1h 10m | Hits:  43%/885   
      🟩 12.5               Pass: 100%/2   | Total:  2h 18m | Avg:  1h 09m | Max:  1h 10m
      🟩 12.6               Pass: 100%/31  | Total: 19h 47m | Avg: 38m 17s | Max:  1h 18m | Hits:  43%/2655  
    🟩 cudacxx
      🟩 ClangCUDA18        Pass: 100%/2   | Total:  1h 47m | Avg: 53m 42s | Max: 54m 50s
      🟩 nvcc12.0           Pass: 100%/5   | Total:  3h 45m | Avg: 45m 08s | Max:  1h 10m | Hits:  43%/885   
      🟩 nvcc12.5           Pass: 100%/2   | Total:  2h 18m | Avg:  1h 09m | Max:  1h 10m
      🟩 nvcc12.6           Pass: 100%/29  | Total: 17h 59m | Avg: 37m 14s | Max:  1h 18m | Hits:  43%/2655  
    🟩 cudacxx_family
      🟩 ClangCUDA          Pass: 100%/2   | Total:  1h 47m | Avg: 53m 42s | Max: 54m 50s
      🟩 nvcc               Pass: 100%/36  | Total:  1d 00h | Avg: 40m 06s | Max:  1h 18m | Hits:  43%/3540  
    🟩 cxx
      🟩 Clang14            Pass: 100%/4   | Total:  2h 39m | Avg: 39m 51s | Max: 42m 39s
      🟩 Clang15            Pass: 100%/1   | Total: 37m 08s | Avg: 37m 08s | Max: 37m 08s
      🟩 Clang16            Pass: 100%/1   | Total: 40m 41s | Avg: 40m 41s | Max: 40m 41s
      🟩 Clang17            Pass: 100%/1   | Total: 37m 07s | Avg: 37m 07s | Max: 37m 07s
      🟩 Clang18            Pass: 100%/7   | Total:  4h 40m | Avg: 40m 03s | Max: 54m 50s
      🟩 GCC7               Pass: 100%/2   | Total:  1h 17m | Avg: 38m 42s | Max: 38m 48s
      🟩 GCC8               Pass: 100%/1   | Total: 40m 19s | Avg: 40m 19s | Max: 40m 19s
      🟩 GCC9               Pass: 100%/2   | Total:  1h 19m | Avg: 39m 51s | Max: 40m 58s
      🟩 GCC10              Pass: 100%/1   | Total: 37m 04s | Avg: 37m 04s | Max: 37m 04s
      🟩 GCC11              Pass: 100%/1   | Total: 42m 30s | Avg: 42m 30s | Max: 42m 30s
      🟩 GCC12              Pass: 100%/3   | Total:  1h 13m | Avg: 24m 35s | Max: 39m 14s
      🟩 GCC13              Pass: 100%/8   | Total:  3h 41m | Avg: 27m 42s | Max: 45m 21s
      🟩 MSVC14.29          Pass: 100%/2   | Total:  2h 16m | Avg:  1h 08m | Max:  1h 10m | Hits:  43%/1770  
      🟩 MSVC14.39          Pass: 100%/2   | Total:  2h 29m | Avg:  1h 14m | Max:  1h 18m | Hits:  43%/1770  
      🟩 NVHPC24.7          Pass: 100%/2   | Total:  2h 18m | Avg:  1h 09m | Max:  1h 10m
    🟩 cxx_family
      🟩 Clang              Pass: 100%/14  | Total:  9h 14m | Avg: 39m 37s | Max: 54m 50s
      🟩 GCC                Pass: 100%/18  | Total:  9h 32m | Avg: 31m 48s | Max: 45m 21s
      🟩 MSVC               Pass: 100%/4   | Total:  4h 45m | Avg:  1h 11m | Max:  1h 18m | Hits:  43%/3540  
      🟩 NVHPC              Pass: 100%/2   | Total:  2h 18m | Avg:  1h 09m | Max:  1h 10m
    🟩 gpu
      🟩 h100               Pass: 100%/2   | Total: 34m 31s | Avg: 17m 15s | Max: 19m 26s
      🟩 v100               Pass: 100%/36  | Total:  1d 01h | Avg: 42m 08s | Max:  1h 18m | Hits:  43%/3540  
    🟩 jobs
      🟩 Build              Pass: 100%/31  | Total: 23h 22m | Avg: 45m 13s | Max:  1h 18m | Hits:  43%/3540  
      🟩 DeviceLaunch       Pass: 100%/1   | Total: 21m 07s | Avg: 21m 07s | Max: 21m 07s
      🟩 GraphCapture       Pass: 100%/1   | Total: 15m 25s | Avg: 15m 25s | Max: 15m 25s
      🟩 HostLaunch         Pass: 100%/3   | Total:  1h 01m | Avg: 20m 38s | Max: 23m 17s
      🟩 TestGPU            Pass: 100%/2   | Total: 51m 03s | Avg: 25m 31s | Max: 26m 03s
    🟩 sm
      🟩 90                 Pass: 100%/2   | Total: 34m 31s | Avg: 17m 15s | Max: 19m 26s
      🟩 90a                Pass: 100%/1   | Total: 15m 19s | Avg: 15m 19s | Max: 15m 19s
    🟩 std
      🟩 17                 Pass: 100%/14  | Total: 11h 26m | Avg: 49m 01s | Max:  1h 10m | Hits:  43%/2655  
      🟩 20                 Pass: 100%/24  | Total: 14h 25m | Avg: 36m 02s | Max:  1h 18m | Hits:  43%/885   
    
  • 🟩 thrust: Pass: 100%/37 | Total: 14h 16m | Avg: 23m 09s | Max: 1h 12m | Hits: 174%/9180

    🟩 cmake_options
      🟩 -DTHRUST_DISPATCH_TYPE=Force32bit Pass: 100%/2   | Total: 43m 47s | Avg: 21m 53s | Max: 32m 19s
    🟩 cpu
      🟩 amd64              Pass: 100%/35  | Total: 13h 52m | Avg: 23m 46s | Max:  1h 12m | Hits: 174%/9180  
      🟩 arm64              Pass: 100%/2   | Total: 24m 34s | Avg: 12m 17s | Max: 12m 49s
    🟩 ctk
      🟩 12.0               Pass: 100%/5   | Total:  1h 56m | Avg: 23m 23s | Max:  1h 04m | Hits: 121%/1836  
      🟩 12.5               Pass: 100%/2   | Total:  2h 16m | Avg:  1h 08m | Max:  1h 09m
      🟩 12.6               Pass: 100%/30  | Total: 10h 03m | Avg: 20m 06s | Max:  1h 12m | Hits: 187%/7344  
    🟩 cudacxx
      🟩 ClangCUDA18        Pass: 100%/2   | Total: 24m 21s | Avg: 12m 10s | Max: 12m 11s
      🟩 nvcc12.0           Pass: 100%/5   | Total:  1h 56m | Avg: 23m 23s | Max:  1h 04m | Hits: 121%/1836  
      🟩 nvcc12.5           Pass: 100%/2   | Total:  2h 16m | Avg:  1h 08m | Max:  1h 09m
      🟩 nvcc12.6           Pass: 100%/28  | Total:  9h 38m | Avg: 20m 40s | Max:  1h 12m | Hits: 187%/7344  
    🟩 cudacxx_family
      🟩 ClangCUDA          Pass: 100%/2   | Total: 24m 21s | Avg: 12m 10s | Max: 12m 11s
      🟩 nvcc               Pass: 100%/35  | Total: 13h 52m | Avg: 23m 47s | Max:  1h 12m | Hits: 174%/9180  
    🟩 cxx
      🟩 Clang14            Pass: 100%/4   | Total: 54m 08s | Avg: 13m 32s | Max: 13m 53s
      🟩 Clang15            Pass: 100%/1   | Total: 13m 06s | Avg: 13m 06s | Max: 13m 06s
      🟩 Clang16            Pass: 100%/1   | Total: 12m 15s | Avg: 12m 15s | Max: 12m 15s
      🟩 Clang17            Pass: 100%/1   | Total: 13m 04s | Avg: 13m 04s | Max: 13m 04s
      🟩 Clang18            Pass: 100%/7   | Total:  1h 42m | Avg: 14m 38s | Max: 31m 59s
      🟩 GCC7               Pass: 100%/2   | Total: 26m 53s | Avg: 13m 26s | Max: 13m 47s
      🟩 GCC8               Pass: 100%/1   | Total: 13m 15s | Avg: 13m 15s | Max: 13m 15s
      🟩 GCC9               Pass: 100%/2   | Total: 25m 09s | Avg: 12m 34s | Max: 12m 54s
      🟩 GCC10              Pass: 100%/1   | Total: 12m 32s | Avg: 12m 32s | Max: 12m 32s
      🟩 GCC11              Pass: 100%/1   | Total: 13m 52s | Avg: 13m 52s | Max: 13m 52s
      🟩 GCC12              Pass: 100%/1   | Total: 13m 47s | Avg: 13m 47s | Max: 13m 47s
      🟩 GCC13              Pass: 100%/8   | Total:  1h 57m | Avg: 14m 40s | Max: 32m 19s
      🟩 MSVC14.29          Pass: 100%/2   | Total:  2h 08m | Avg:  1h 04m | Max:  1h 04m | Hits: 131%/3672  
      🟩 MSVC14.39          Pass: 100%/3   | Total:  2h 53m | Avg: 57m 59s | Max:  1h 12m | Hits: 203%/5508  
      🟩 NVHPC24.7          Pass: 100%/2   | Total:  2h 16m | Avg:  1h 08m | Max:  1h 09m
    🟩 cxx_family
      🟩 Clang              Pass: 100%/14  | Total:  3h 14m | Avg: 13m 55s | Max: 31m 59s
      🟩 GCC                Pass: 100%/16  | Total:  3h 42m | Avg: 13m 55s | Max: 32m 19s
      🟩 MSVC               Pass: 100%/5   | Total:  5h 02m | Avg:  1h 00m | Max:  1h 12m | Hits: 174%/9180  
      🟩 NVHPC              Pass: 100%/2   | Total:  2h 16m | Avg:  1h 08m | Max:  1h 09m
    🟩 gpu
      🟩 v100               Pass: 100%/37  | Total: 14h 16m | Avg: 23m 09s | Max:  1h 12m | Hits: 174%/9180  
    🟩 jobs
      🟩 Build              Pass: 100%/31  | Total: 12h 03m | Avg: 23m 19s | Max:  1h 12m | Hits: 126%/7344  
      🟩 TestCPU            Pass: 100%/3   | Total: 51m 54s | Avg: 17m 18s | Max: 36m 01s | Hits: 365%/1836  
      🟩 TestGPU            Pass: 100%/3   | Total:  1h 21m | Avg: 27m 17s | Max: 32m 19s
    🟩 sm
      🟩 90a                Pass: 100%/1   | Total:  8m 37s | Avg:  8m 37s | Max:  8m 37s
    🟩 std
      🟩 17                 Pass: 100%/14  | Total:  6h 32m | Avg: 28m 02s | Max:  1h 09m | Hits: 128%/5508  
      🟩 20                 Pass: 100%/21  | Total:  7h 00m | Avg: 20m 01s | Max:  1h 12m | Hits: 243%/3672  
    
  • 🟩 cccl_c_parallel: Pass: 100%/2 | Total: 10m 07s | Avg: 5m 03s | Max: 8m 02s

    🟩 cpu
      🟩 amd64              Pass: 100%/2   | Total: 10m 07s | Avg:  5m 03s | Max:  8m 02s
    🟩 ctk
      🟩 12.6               Pass: 100%/2   | Total: 10m 07s | Avg:  5m 03s | Max:  8m 02s
    🟩 cudacxx
      🟩 nvcc12.6           Pass: 100%/2   | Total: 10m 07s | Avg:  5m 03s | Max:  8m 02s
    🟩 cudacxx_family
      🟩 nvcc               Pass: 100%/2   | Total: 10m 07s | Avg:  5m 03s | Max:  8m 02s
    🟩 cxx
      🟩 GCC13              Pass: 100%/2   | Total: 10m 07s | Avg:  5m 03s | Max:  8m 02s
    🟩 cxx_family
      🟩 GCC                Pass: 100%/2   | Total: 10m 07s | Avg:  5m 03s | Max:  8m 02s
    🟩 gpu
      🟩 v100               Pass: 100%/2   | Total: 10m 07s | Avg:  5m 03s | Max:  8m 02s
    🟩 jobs
      🟩 Build              Pass: 100%/1   | Total:  2m 05s | Avg:  2m 05s | Max:  2m 05s
      🟩 Test               Pass: 100%/1   | Total:  8m 02s | Avg:  8m 02s | Max:  8m 02s
    
  • 🟩 python: Pass: 100%/1 | Total: 47m 33s | Avg: 47m 33s | Max: 47m 33s

    🟩 cpu
      🟩 amd64              Pass: 100%/1   | Total: 47m 33s | Avg: 47m 33s | Max: 47m 33s
    🟩 ctk
      🟩 12.6               Pass: 100%/1   | Total: 47m 33s | Avg: 47m 33s | Max: 47m 33s
    🟩 cudacxx
      🟩 nvcc12.6           Pass: 100%/1   | Total: 47m 33s | Avg: 47m 33s | Max: 47m 33s
    🟩 cudacxx_family
      🟩 nvcc               Pass: 100%/1   | Total: 47m 33s | Avg: 47m 33s | Max: 47m 33s
    🟩 cxx
      🟩 GCC13              Pass: 100%/1   | Total: 47m 33s | Avg: 47m 33s | Max: 47m 33s
    🟩 cxx_family
      🟩 GCC                Pass: 100%/1   | Total: 47m 33s | Avg: 47m 33s | Max: 47m 33s
    🟩 gpu
      🟩 v100               Pass: 100%/1   | Total: 47m 33s | Avg: 47m 33s | Max: 47m 33s
    🟩 jobs
      🟩 Test               Pass: 100%/1   | Total: 47m 33s | Avg: 47m 33s | Max: 47m 33s
    

👃 Inspect Changes

Modifications in project?

Project
CCCL Infrastructure
libcu++
+/- CUB
Thrust
CUDA Experimental
python
CCCL C Parallel Library
Catch2Helper

Modifications in project or dependencies?

Project
CCCL Infrastructure
libcu++
+/- CUB
+/- Thrust
CUDA Experimental
+/- python
+/- CCCL C Parallel Library
+/- Catch2Helper

🏃‍ Runner counts (total jobs: 78)

# Runner
53 linux-amd64-cpu16
11 linux-amd64-gpu-v100-latest-1
9 windows-amd64-cpu16
4 linux-arm64-cpu16
1 linux-amd64-gpu-h100-latest-1-testing

@bernhardmgruber bernhardmgruber enabled auto-merge (squash) January 21, 2025 16:08
@bernhardmgruber bernhardmgruber merged commit 32f41dd into NVIDIA:main Jan 21, 2025
92 of 95 checks passed
Copy link
Contributor

Git push to origin failed for branch/2.8.x with exitcode 128

@bernhardmgruber bernhardmgruber deleted the depr_cub_macro branch January 21, 2025 16:14
bernhardmgruber added a commit to bernhardmgruber/cccl that referenced this pull request Jan 21, 2025
@fbusato
Copy link
Contributor

fbusato commented Jan 21, 2025

would be useful to add LIBCUDACXX_COMP_DEPR_IMPL to the deprecated macros?

@bernhardmgruber
Copy link
Contributor Author

bernhardmgruber commented Jan 21, 2025

would be useful to add LIBCUDACXX_COMP_DEPR_IMPL to the deprecated macros?

Sure, add it to the list: #101

davebayer pushed a commit to davebayer/cccl that referenced this pull request Jan 22, 2025
davebayer added a commit to davebayer/cccl that referenced this pull request Jan 22, 2025
update docs

update docs

add `memcmp`, `memmove` and `memchr` implementations

implement tests

Use cuda::std::min/max in Thrust (NVIDIA#3364)

Implement `cuda::std::numeric_limits` for `__half` and `__nv_bfloat16` (NVIDIA#3361)

* implement `cuda::std::numeric_limits` for `__half` and `__nv_bfloat16`

Cleanup util_arch (NVIDIA#2773)

Deprecate thrust::null_type (NVIDIA#3367)

Deprecate cub::DeviceSpmv (NVIDIA#3320)

Fixes: NVIDIA#896

Improves `DeviceSegmentedSort` test run time for large number of items and segments (NVIDIA#3246)

* fixes segment offset generation

* switches to analytical verification

* switches to analytical verification for pairs

* fixes spelling

* adds tests for large number of segments

* fixes narrowing conversion in tests

* addresses review comments

* fixes includes

Compile basic infra test with C++17 (NVIDIA#3377)

Adds support for large number of items and large number of segments to `DeviceSegmentedSort` (NVIDIA#3308)

* fixes segment offset generation

* switches to analytical verification

* switches to analytical verification for pairs

* addresses review comments

* introduces segment offset type

* adds tests for large number of segments

* adds support for large number of segments

* drops segment offset type

* fixes thrust namespace

* removes about-to-be-deprecated cub iterators

* no exec specifier on defaulted ctor

* fixes gcc7 linker error

* uses local_segment_index_t throughout

* determine offset type based on type returned by segment iterator begin/end iterators

* minor style improvements

Exit with error when RAPIDS CI fails. (NVIDIA#3385)

cuda.parallel: Support structured types as algorithm inputs (NVIDIA#3218)

* Introduce gpu_struct decorator and typing

* Enable `reduce` to accept arrays of structs as inputs

* Add test for reducing arrays-of-struct

* Update documentation

* Use a numpy array rather than ctypes object

* Change zeros -> empty for output array and temp storage

* Add a TODO for typing GpuStruct

* Documentation udpates

* Remove test_reduce_struct_type from test_reduce.py

* Revert to `to_cccl_value()` accepting ndarray + GpuStruct

* Bump copyrights

---------

Co-authored-by: Ashwin Srinath <[email protected]>

Deprecate thrust::async (NVIDIA#3324)

Fixes: NVIDIA#100

Review/Deprecate CUB `util.ptx` for CCCL 2.x (NVIDIA#3342)

Fix broken `_CCCL_BUILTIN_ASSUME` macro (NVIDIA#3314)

* add compiler-specific path
* fix device code path
* add _CCC_ASSUME

Deprecate thrust::numeric_limits (NVIDIA#3366)

Replace `typedef` with `using` in libcu++ (NVIDIA#3368)

Deprecate thrust::optional (NVIDIA#3307)

Fixes: NVIDIA#3306

Upgrade to Catch2 3.8  (NVIDIA#3310)

Fixes: NVIDIA#1724

refactor `<cuda/std/cstdint>` (NVIDIA#3325)

Co-authored-by: Bernhard Manfred Gruber <[email protected]>

Update CODEOWNERS (NVIDIA#3331)

* Update CODEOWNERS

* Update CODEOWNERS

* Update CODEOWNERS

* [pre-commit.ci] auto code formatting

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Fix sign-compare warning (NVIDIA#3408)

Implement more cmath functions to be usable on host and device (NVIDIA#3382)

* Implement more cmath functions to be usable on host and device

* Implement math roots functions

* Implement exponential functions

Redefine and deprecate thrust::remove_cvref (NVIDIA#3394)

* Redefine and deprecate thrust::remove_cvref

Co-authored-by: Michael Schellenberger Costa <[email protected]>

Fix assert definition for NVHPC due to constexpr issues (NVIDIA#3418)

NVHPC cannot decide at compile time where the code would run so _CCCL_ASSERT within a constexpr function breaks it.

Fix this by always using the host definition which should also work on device.

Fixes NVIDIA#3411

Extend CUB reduce benchmarks (NVIDIA#3401)

* Rename max.cu to custom.cu, since it uses a custom operator
* Extend types covered my min.cu to all fundamental types
* Add some notes on how to collect tuning parameters

Fixes: NVIDIA#3283

Update upload-pages-artifact to v3 (NVIDIA#3423)

* Update upload-pages-artifact to v3

* Empty commit

---------

Co-authored-by: Ashwin Srinath <[email protected]>

Replace and deprecate thrust::cuda_cub::terminate (NVIDIA#3421)

`std::linalg` accessors and `transposed_layout` (NVIDIA#2962)

Add round up/down to multiple (NVIDIA#3234)

[FEA]: Introduce Python module with CCCL headers (NVIDIA#3201)

* Add cccl/python/cuda_cccl directory and use from cuda_parallel, cuda_cooperative

* Run `copy_cccl_headers_to_aude_include()` before `setup()`

* Create python/cuda_cccl/cuda/_include/__init__.py, then simply import cuda._include to find the include path.

* Add cuda.cccl._version exactly as for cuda.cooperative and cuda.parallel

* Bug fix: cuda/_include only exists after shutil.copytree() ran.

* Use `f"cuda-cccl @ file://{cccl_path}/python/cuda_cccl"` in setup.py

* Remove CustomBuildCommand, CustomWheelBuild in cuda_parallel/setup.py (they are equivalent to the default functions)

* Replace := operator (needs Python 3.8+)

* Fix oversights: remove `pip3 install ./cuda_cccl` lines from README.md

* Restore original README.md: `pip3 install -e` now works on first pass.

* cuda_cccl/README.md: FOR INTERNAL USE ONLY

* Remove `$pymajor.$pyminor.` prefix in cuda_cccl _version.py (as suggested under NVIDIA#3201 (comment))

Command used: ci/update_version.sh 2 8 0

* Modernize pyproject.toml, setup.py

Trigger for this change:

* NVIDIA#3201 (comment)

* NVIDIA#3201 (comment)

* Install CCCL headers under cuda.cccl.include

Trigger for this change:

* NVIDIA#3201 (comment)

Unexpected accidental discovery: cuda.cooperative unit tests pass without CCCL headers entirely.

* Factor out cuda_cccl/cuda/cccl/include_paths.py

* Reuse cuda_cccl/cuda/cccl/include_paths.py from cuda_cooperative

* Add missing Copyright notice.

* Add missing __init__.py (cuda.cccl)

* Add `"cuda.cccl"` to `autodoc.mock_imports`

* Move cuda.cccl.include_paths into function where it is used. (Attempt to resolve Build and Verify Docs failure.)

* Add # TODO: move this to a module-level import

* Modernize cuda_cooperative/pyproject.toml, setup.py

* Convert cuda_cooperative to use hatchling as build backend.

* Revert "Convert cuda_cooperative to use hatchling as build backend."

This reverts commit 61637d6.

* Move numpy from [build-system] requires -> [project] dependencies

* Move pyproject.toml [project] dependencies -> setup.py install_requires, to be able to use CCCL_PATH

* Remove copy_license() and use license_files=["../../LICENSE"] instead.

* Further modernize cuda_cccl/setup.py to use pathlib

* Trivial simplifications in cuda_cccl/pyproject.toml

* Further simplify cuda_cccl/pyproject.toml, setup.py: remove inconsequential code

* Make cuda_cooperative/pyproject.toml more similar to cuda_cccl/pyproject.toml

* Add taplo-pre-commit to .pre-commit-config.yaml

* taplo-pre-commit auto-fixes

* Use pathlib in cuda_cooperative/setup.py

* CCCL_PYTHON_PATH in cuda_cooperative/setup.py

* Modernize cuda_parallel/pyproject.toml, setup.py

* Use pathlib in cuda_parallel/setup.py

* Add `# TOML lint & format` comment.

* Replace MANIFEST.in with `[tool.setuptools.package-data]` section in pyproject.toml

* Use pathlib in cuda/cccl/include_paths.py

* pre-commit autoupdate (EXCEPT clang-format, which was manually restored)

* Fixes after git merge main

* Resolve warning: AttributeError: '_Reduce' object has no attribute 'build_result'

```
=========================================================================== warnings summary ===========================================================================
tests/test_reduce.py::test_reduce_non_contiguous
  /home/coder/cccl/python/devenv/lib/python3.12/site-packages/_pytest/unraisableexception.py:85: PytestUnraisableExceptionWarning: Exception ignored in: <function _Reduce.__del__ at 0x7bf123139080>

  Traceback (most recent call last):
    File "/home/coder/cccl/python/cuda_parallel/cuda/parallel/experimental/algorithms/reduce.py", line 132, in __del__
      bindings.cccl_device_reduce_cleanup(ctypes.byref(self.build_result))
                                                       ^^^^^^^^^^^^^^^^^
  AttributeError: '_Reduce' object has no attribute 'build_result'

    warnings.warn(pytest.PytestUnraisableExceptionWarning(msg))

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
============================================================= 1 passed, 93 deselected, 1 warning in 0.44s ==============================================================
```

* Move `copy_cccl_headers_to_cuda_cccl_include()` functionality to `class CustomBuildPy`

* Introduce cuda_cooperative/constraints.txt

* Also add cuda_parallel/constraints.txt

* Add `--constraint constraints.txt` in ci/test_python.sh

* Update Copyright dates

* Switch to https://github.com/ComPWA/taplo-pre-commit (the other repo has been archived by the owner on Jul 1, 2024)

For completeness: The other repo took a long time to install into the pre-commit cache; so long it lead to timeouts in the CCCL CI.

* Remove unused cuda_parallel jinja2 dependency (noticed by chance).

* Remove constraints.txt files, advertise running `pip install cuda-cccl` first instead.

* Make cuda_cooperative, cuda_parallel testing completely independent.

* Run only test_python.sh [skip-rapids][skip-matx][skip-docs][skip-vdc]

* Try using another runner (because V100 runners seem to be stuck) [skip-rapids][skip-matx][skip-docs][skip-vdc]

* Fix sign-compare warning (NVIDIA#3408) [skip-rapids][skip-matx][skip-docs][skip-vdc]

* Revert "Try using another runner (because V100 runners seem to be stuck) [skip-rapids][skip-matx][skip-docs][skip-vdc]"

This reverts commit ea33a21.

Error message: NVIDIA#3201 (comment)

* Try using A100 runner (because V100 runners still seem to be stuck) [skip-rapids][skip-matx][skip-docs][skip-vdc]

* Also show cuda-cooperative site-packages, cuda-parallel site-packages (after pip install) [skip-rapids][skip-matx][skip-docs][skip-vdc]

* Try using l4 runner (because V100 runners still seem to be stuck) [skip-rapids][skip-matx][skip-docs][skip-vdc]

* Restore original ci/matrix.yaml [skip-rapids]

* Use for loop in test_python.sh to avoid code duplication.

* Run only test_python.sh [skip-rapids][skip-matx][skip-docs][skip-vdc][skip pre-commit.ci]

* Comment out taplo-lint in pre-commit config [skip-rapids][skip-matx][skip-docs][skip-vdc]

* Revert "Run only test_python.sh [skip-rapids][skip-matx][skip-docs][skip-vdc][skip pre-commit.ci]"

This reverts commit ec206fd.

* Implement suggestion by @shwina (NVIDIA#3201 (review))

* Address feedback by @leofang

---------

Co-authored-by: Bernhard Manfred Gruber <[email protected]>

cuda.parallel: Add optional stream argument to reduce_into() (NVIDIA#3348)

* Add optional stream argument to reduce_into()

* Add tests to check for reduce_into() stream behavior

* Move protocol related utils to separate file and rework __cuda_stream__ error messages

* Fix synchronization issue in stream test and add one more invalid stream test case

* Rename cuda stream validation function after removing leading underscore

* Unpack values from __cuda_stream__ instead of indexing

* Fix linting errors

* Handle TypeError when unpacking invalid __cuda_stream__ return

* Use stream to allocate cupy memory in new stream test

Upgrade to actions/deploy-pages@v4 (from v2), as suggested by @leofang (NVIDIA#3434)

Deprecate `cub::{min, max}` and replace internal uses with those from libcu++ (NVIDIA#3419)

* Deprecate `cub::{min, max}` and replace internal uses with those from libcu++

Fixes NVIDIA#3404

Fix CI issues (NVIDIA#3443)

Remove deprecated `cub::min` (NVIDIA#3450)

* Remove deprecated `cuda::{min,max}`

* Drop unused `thrust::remove_cvref` file

Fix typo in builtin (NVIDIA#3451)

Moves agents to `detail::<algorithm_name>` namespace (NVIDIA#3435)

uses unsigned offset types in thrust's scan dispatch (NVIDIA#3436)

Default transform_iterator's copy ctor (NVIDIA#3395)

Fixes: NVIDIA#2393

Turn C++ dialect warning into error (NVIDIA#3453)

Uses unsigned offset types in thrust's sort algorithm calling into `DispatchMergeSort` (NVIDIA#3437)

* uses thrust's dynamic dispatch for merge_sort

* [pre-commit.ci] auto code formatting

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Refactor allocator handling of contiguous_storage (NVIDIA#3050)

Co-authored-by: Michael Schellenberger Costa <[email protected]>

Drop thrust::detail::integer_traits (NVIDIA#3391)

Add cuda::is_floating_point supporting half and bfloat (NVIDIA#3379)

Co-authored-by: Michael Schellenberger Costa <[email protected]>

Improve docs of std headers (NVIDIA#3416)

Drop C++11 and C++14 support for all of cccl (NVIDIA#3417)

* Drop C++11 and C++14 support for all of cccl

---------

Co-authored-by: Bernhard Manfred Gruber <[email protected]>

Deprecate a few CUB macros (NVIDIA#3456)

Deprecate thrust universal iterator categories (NVIDIA#3461)

Fix launch args order (NVIDIA#3465)

Add `--extended-lambda` to the list of removed clangd flags (NVIDIA#3432)

add `_CCCL_HAS_NVFP8` macro (NVIDIA#3429)

Add `_CCCL_BUILTIN_PREFETCH` (NVIDIA#3433)

Drop universal iterator categories (NVIDIA#3474)

Ensure that headers in `<cuda/*>` can be build with a C++ only compiler (NVIDIA#3472)

Specialize __is_extended_floating_point for FP8 types (NVIDIA#3470)

Also ensure that we actually can enable FP8 due to FP16 and BF16 requirements

Co-authored-by: Michael Schellenberger Costa <[email protected]>

Moves CUB kernel entry points to a detail namespace (NVIDIA#3468)

* moves emptykernel to detail ns

* second batch

* third batch

* fourth batch

* fixes cuda parallel

* concatenates nested namespaces

Deprecate block/warp algo specializations (NVIDIA#3455)

Fixes: NVIDIA#3409

Refactor CUB's util_debug (NVIDIA#3345)
davebayer pushed a commit to davebayer/cccl that referenced this pull request Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport branch/2.8.x cub For all items related to CUB
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants