Skip to content

Commit

Permalink
Resolve "incorrect-portions-license" errors: add no license lint to t…
Browse files Browse the repository at this point in the history
…op of GPU files with both licenses (#3965)

Summary:
Pull Request resolved: #3965

End goal: add some lint to prevent files landing without any header. This creates churn for the oncall. Example is the last diff in this stack: D64488600

This is the first diff in enabling linting. I wanted the diff which updates all files to be just the automated command and the .toml config files changes, so this diff is a preparation step.

This particular diff solves: we get errors due to having both NVidia and FB license comment headers in the same files. See the errors in D64429711 under "linter-coverage-verification" signal.

Context on linting: https://fb.workplace.com/groups/lintqa/posts/3762373340693743/?comment_id=3762915930639484&reply_comment_id=3763044163959994

Per OSS, we cannot remove the Nvidia notice: https://fb.workplace.com/groups/osssupport/posts/27179779734977284/?comment_id=27181801724775085&reply_comment_id=27181818148106776

So, we add the lint ignore here.

Reviewed By: asadoughi

Differential Revision: D64481766

fbshipit-source-id: 0468104a095831cac7dd5bf3c519cfb5a2ee1575
  • Loading branch information
Michael Norris authored and facebook-github-bot committed Oct 21, 2024
1 parent f9a01c6 commit fd6d784
Show file tree
Hide file tree
Showing 26 changed files with 28 additions and 2 deletions.
1 change: 1 addition & 0 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
# @lint-ignore-every LICENSELINT
# Copyright (c) Facebook, Inc. and its affiliates.
# All rights reserved.
#
Expand Down
3 changes: 2 additions & 1 deletion cmake/FindMKL.cmake
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
# Copyright (c) Facebook, Inc. and its affiliates.
# @lint-ignore-every LICENSELINT
# Copyright (c) Meta Platforms, Inc. and affiliates.
# All rights reserved.
#
# This source code is licensed under the BSD-style license found in the
Expand Down
3 changes: 2 additions & 1 deletion cmake/link_to_faiss_lib.cmake
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
# Copyright (c) Facebook, Inc. and its affiliates.
# @lint-ignore-every LICENSELINT
# Copyright (c) Meta Platforms, Inc. and affiliates.
# All rights reserved.
#
# This source code is licensed under the BSD-style license found in the
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/GpuDistance.cu
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/GpuIndex.h
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/GpuIndexCagra.cu
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/GpuIndexCagra.h
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/GpuResources.cpp
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/GpuResources.h
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/StandardGpuResources.cpp
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/StandardGpuResources.h
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/impl/FlatIndex.cu
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/impl/FlatIndex.cuh
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/impl/RaftCagra.cu
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/impl/RaftCagra.cuh
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/impl/RaftFlatIndex.cu
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/impl/RaftFlatIndex.cuh
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/impl/RaftIVFFlat.cu
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/impl/RaftIVFFlat.cuh
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/impl/RaftIVFPQ.cu
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/impl/RaftIVFPQ.cuh
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/test/TestGpuDistance.cu
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/test/TestGpuIndexCagra.cu
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/test/TestGpuIndexIVFFlat.cpp
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/utils/RaftUtils.cu
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down
1 change: 1 addition & 0 deletions faiss/gpu/utils/RaftUtils.h
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
// @lint-ignore-every LICENSELINT
/**
* Copyright (c) Facebook, Inc. and its affiliates.
*
Expand Down

0 comments on commit fd6d784

Please sign in to comment.