Skip to content

Replace proprietary license headers with Apache-2.0#19

Merged
ivanbasov merged 5 commits into
NVIDIA:mainfrom
ivanbasov:worktree-headers
Mar 23, 2026
Merged

Replace proprietary license headers with Apache-2.0#19
ivanbasov merged 5 commits into
NVIDIA:mainfrom
ivanbasov:worktree-headers

Conversation

@ivanbasov
Copy link
Copy Markdown
Member

@ivanbasov ivanbasov commented Mar 20, 2026

Summary

  • Replace LicenseRef-NvidiaProprietary SPDX headers with Apache-2.0 across all 71 tracked source files
  • Update spdx_headers.py to generate Apache-2.0 headers and perform in-place replacement of old proprietary headers (preserving the existing copyright year)
  • CI spdx-header-check job passes with the new headers

Test plan

  • CI spdx-header-check passes (python3 code/scripts/spdx_headers.py --check)
  • All other CI jobs pass

🤖 Generated with Claude Code

@ivanbasov ivanbasov requested a review from bmhowe23 March 20, 2026 03:32
@ivanbasov ivanbasov force-pushed the worktree-headers branch 2 times, most recently from d5a3f33 to 79c980b Compare March 23, 2026 17:34
ivanbasov and others added 3 commits March 23, 2026 10:38
Update all SPDX headers from LicenseRef-NvidiaProprietary to Apache-2.0
across all 70 tracked source files. Also updates spdx_headers.py to
generate Apache-2.0 headers and replace old proprietary headers in-place.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Files added by PRs #13, #14, and #17 still carried the proprietary
LicenseRef-NvidiaProprietary header. Replace with Apache-2.0 to match
the rest of the codebase after the header migration.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
ivanbasov and others added 2 commits March 23, 2026 10:50
The first rebase used --theirs to resolve header conflicts, which took
the old PR branch content instead of main's newer content for 5 files.
Restore from upstream/main and apply Apache-2.0 header correctly.

Affected files:
- code/qec/noise_model.py
- code/qec/surface_code/homological_equivalence_torch.py
- code/tests/mid/test_homological_equivalence.py
- code/tests/test_noise_model.py
- code/training/train.py

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
PR #18 removed the unused v2 model architecture. Drop the corresponding
test class and import to fix the ImportError in CI.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ivanbasov ivanbasov merged commit b5ff9eb into NVIDIA:main Mar 23, 2026
12 checks passed
@ivanbasov ivanbasov deleted the worktree-headers branch March 23, 2026 21:16
ivanbasov added a commit that referenced this pull request Apr 10, 2026
* Replace proprietary license headers with Apache-2.0

Update all SPDX headers from LicenseRef-NvidiaProprietary to Apache-2.0
across all 70 tracked source files. Also updates spdx_headers.py to
generate Apache-2.0 headers and replace old proprietary headers in-place.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(headers): apply Apache-2.0 headers to files added after branch cut

Files added by PRs #13, #14, and #17 still carried the proprietary
LicenseRef-NvidiaProprietary header. Replace with Apache-2.0 to match
the rest of the codebase after the header migration.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* style: apply YAPF formatting after header replacement

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(headers): restore full file content truncated during rebase

The first rebase used --theirs to resolve header conflicts, which took
the old PR branch content instead of main's newer content for 5 files.
Restore from upstream/main and apply Apache-2.0 header correctly.

Affected files:
- code/qec/noise_model.py
- code/qec/surface_code/homological_equivalence_torch.py
- code/tests/mid/test_homological_equivalence.py
- code/tests/test_noise_model.py
- code/training/train.py

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(test): remove PreDecoderModelMemory_v2 test removed by PR #18

PR #18 removed the unused v2 model architecture. Drop the corresponding
test class and import to fix the ImportError in CI.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
ivanbasov added a commit that referenced this pull request Apr 10, 2026
* Replace proprietary license headers with Apache-2.0

Update all SPDX headers from LicenseRef-NvidiaProprietary to Apache-2.0
across all 70 tracked source files. Also updates spdx_headers.py to
generate Apache-2.0 headers and replace old proprietary headers in-place.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(headers): apply Apache-2.0 headers to files added after branch cut

Files added by PRs #13, #14, and #17 still carried the proprietary
LicenseRef-NvidiaProprietary header. Replace with Apache-2.0 to match
the rest of the codebase after the header migration.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* style: apply YAPF formatting after header replacement

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(headers): restore full file content truncated during rebase

The first rebase used --theirs to resolve header conflicts, which took
the old PR branch content instead of main's newer content for 5 files.
Restore from upstream/main and apply Apache-2.0 header correctly.

Affected files:
- code/qec/noise_model.py
- code/qec/surface_code/homological_equivalence_torch.py
- code/tests/mid/test_homological_equivalence.py
- code/tests/test_noise_model.py
- code/training/train.py

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(test): remove PreDecoderModelMemory_v2 test removed by PR #18

PR #18 removed the unused v2 model architecture. Drop the corresponding
test class and import to fix the ImportError in CI.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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