Skip to content

Conversation

@astachowiczhabana
Copy link

@astachowiczhabana astachowiczhabana commented Dec 5, 2025

Introduces fixheaders, a tool for automated license header management across the repository + fixes license issues.

Key Features

  • Multi-format support (Python, C/C++, CUDA, YAML, CMake)
  • YAML-based configuration for default and custom headers
  • Dry-run mode for safe verification
  • Glob pattern matching for file selection

Usage
cd tools/fixheaders; ./run.sh

Pre-Release Process

  1. Run Protex scan
  2. Identify third-party code matches
  3. Add exact copyright headers to appropriate YAML config
  4. Run "cd tools/fixheaders; ./run.sh" to apply/verify headers

Copilot AI review requested due to automatic review settings December 5, 2025 14:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces automated license header management across the repository by implementing a fixheaders tool and applying standardized license headers to files.

Key Changes:

  • Added Intel Corporation copyright headers (2025) with Apache-2.0 license to all source files
  • Included dual licensing for files derived from third-party sources (PyTorch, Torchvision, Codeplay)
  • Applied headers to C/C++, CUDA, YAML, and CMake files

Reviewed changes

Copilot reviewed 300 out of 897 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/ATen/native/xpu/sycl/*.{h,cpp} Added Intel copyright headers to SYCL kernel implementations
src/ATen/native/xpu/mkl/*.{h,cpp} Added Intel copyright headers to oneMKL wrapper implementations
src/ATen/native/xpu/*.cpp Added Intel copyright headers to XPU native operations
src/ATen/native/transformers/**/*.{h,cpp} Added Intel copyright headers with Codeplay dual licensing where applicable
src/ATen/native/sparse/xpu/**/*.{h,cpp} Added Intel copyright headers to sparse tensor implementations
src/ATen/native/quantized/**/*.{h,cpp} Added Intel copyright headers to quantization implementations
src/ATen/native/nested/**/*.cpp Added Intel copyright headers to nested tensor implementations
cmake/**/*.cmake Added Intel copyright headers to CMake configuration files
CMakeLists.txt, src/ATen/CMakeLists.txt Added Intel copyright headers to build configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@riverliuintel riverliuintel left a comment

Choose a reason for hiding this comment

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

@astachowiczhabana
Copy link
Author

astachowiczhabana commented Dec 8, 2025

Hi @riverliuintel. I've updated the Intel license headers according to your suggestion from https://github.com/intel/torch-xpu-ops/blob/main/LICENSE#L190.

Failing TC is:

  • op_transformers,test.test_transformers.TestSDPAXpuOnlyXPU,test_flash_attention_unsupport_bhsd_layout_xpu

It's unrelated to this PR, it was enabled few days ago with this PR: pytorch/pytorch#167057

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants