Skip to content

Use SPDX for all copyright headers#1446

Merged
rapids-bot[bot] merged 10 commits intorapidsai:mainfrom
KyleFromNVIDIA:spdx-copyright
Oct 27, 2025
Merged

Use SPDX for all copyright headers#1446
rapids-bot[bot] merged 10 commits intorapidsai:mainfrom
KyleFromNVIDIA:spdx-copyright

Conversation

@KyleFromNVIDIA
Copy link
Copy Markdown
Member

@KyleFromNVIDIA KyleFromNVIDIA requested review from a team as code owners October 21, 2025 20:49
@KyleFromNVIDIA KyleFromNVIDIA added the improvement Improves an existing functionality label Oct 21, 2025
@KyleFromNVIDIA KyleFromNVIDIA requested review from a team as code owners October 21, 2025 20:49
@KyleFromNVIDIA KyleFromNVIDIA added the non-breaking Introduces a non-breaking change label Oct 21, 2025
@robertmaynard
Copy link
Copy Markdown
Contributor

Files needing changes:

.flake8
conda/recipes/cuvs/recipe.yaml
conda/recipes/libcuvs/recipe.yaml
conda/recipes/cuvs-bench/recipe.yaml
conda/recipes/cuvs-bench-cpu/recipe.yaml
cpp/src/neighbors/detail/faiss_distance_utils.h
.pre-commit-config.yaml

@robertmaynard
Copy link
Copy Markdown
Contributor

Files that need dual AND SPD markup

cpp/src/distance/detail/fused_distance_nn/gemm.h
cpp/src/distance/detail/fused_distance_nn/epilogue.cuh
cpp/src/distance/detail/fused_distance_nn/predicated_tile_iterator_normvec_smem.h
cpp/src/distance/detail/fused_distance_nn/persistent_gemm.h
cpp/src/distance/detail/fused_distance_nn/epilogue_elementwise.cuh
cpp/src/distance/detail/fused_distance_nn/predicated_tile_iterator_reduced_vec.h
cpp/src/distance/detail/fused_distance_nn/custom_epilogue_with_broadcast.h

@KyleFromNVIDIA
Copy link
Copy Markdown
Member Author

Looking at cpp/src/distance/detail/fused_distance_nn/gemm.h, it looks like we own it, and the BSD-3-Clause is left over from the 2017-2023 era. It looks like we simply relicensed this file. I don't see any other ownership. I think we can leave it as Apache-2.0. I'll confirm the others, but I suspect they have the same deal.

@robertmaynard
Copy link
Copy Markdown
Contributor

Looking at cpp/src/distance/detail/fused_distance_nn/gemm.h, it looks like we own it, and the BSD-3-Clause is left over from the 2017-2023 era.

I don't know if we need permission from the cutlass team as that is the original source

Copy link
Copy Markdown
Contributor

@robertmaynard robertmaynard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cpp/cmake/modules/FindAVX.cmake Missing 7 copyright authors

@cjnolet cjnolet moved this from Todo to In Progress in Unstructured Data Processing Oct 22, 2025
Copy link
Copy Markdown
Contributor

@robertmaynard robertmaynard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cpp/src/neighbors/detail/faiss_distance_utils.h is missing SPDX-FileCopyrightText: Copyright (c) Facebook, Inc.

@benfred
Copy link
Copy Markdown
Contributor

benfred commented Oct 27, 2025

/merge

@rapids-bot rapids-bot Bot merged commit c179ccb into rapidsai:main Oct 27, 2025
160 of 164 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Unstructured Data Processing Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants