Skip to content

Remove personal copyright notice from SM90 backward barriers - #434

Merged
vedaanta merged 1 commit into
NVIDIA:developfrom
jiayus-nvidia:remove-jerry-chen-copyright-bwd-barriers
Jul 24, 2026
Merged

Remove personal copyright notice from SM90 backward barriers#434
vedaanta merged 1 commit into
NVIDIA:developfrom
jiayus-nvidia:remove-jerry-chen-copyright-bwd-barriers

Conversation

@jiayus-nvidia

@jiayus-nvidia jiayus-nvidia commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Before submitting

  • I agree to license this contribution under the terms of LICENSE.txt.
  • I ran pre-commit run and committed any formatting changes.

Affected area

FE OSS kernels or CuTeDSL

Summary

Remove the personal Jerry Chen copyright notice from
python/cudnn/deepseek_sparse_attention/utils/sm90/bwd_barriers.py.

The existing multi-author copyright notice and all executable code remain unchanged.

Why

This is a narrowly scoped licensing cleanup ahead of the repository's transition to Apache-2.0.

Related issues

None.

API and compatibility impact

None.

Testing

  • git diff --check passed.
  • Python AST parsing passed.
  • No runtime tests were required because this is a comment-only change.
  • pre-commit was not available in the local environment.

Summary by CodeRabbit

  • Chores
    • Updated copyright attribution and year information.
    • No user-facing functionality or behavior changed.

@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The copyright header in bwd_barriers.py was updated with a new year and author list. No functional code or barrier definitions changed.

Changes

Copyright Header Update

Layer / File(s) Summary
Update copyright header
python/cudnn/deepseek_sparse_attention/utils/sm90/bwd_barriers.py
Replaces the existing copyright line with updated year and author information.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title accurately describes the comment-only cleanup and matches the main change in the PR.
Description check ✅ Passed The description follows the template and includes the affected area, summary, why, issues, impact, and testing.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@NVIDIA-JerryChen

Copy link
Copy Markdown
Contributor

lgtm

@jiayus-nvidia jiayus-nvidia changed the title Remove Jerry Chen copyright notice from SM90 backward barriers Remove personal copyright notice from SM90 backward barriers Jul 24, 2026
@vedaanta
vedaanta merged commit 5fc65ec into NVIDIA:develop Jul 24, 2026
1 check passed
Anerudhan added a commit to Anerudhan/cudnn-frontend that referenced this pull request Jul 29, 2026
…y MIT)

Rebased onto current develop (4f75b1f). Relicenses NVIDIA-authored code from
MIT to Apache-2.0 using the exact NVIDIA OSS SPDX header, while keeping under
MIT every file that carries non-NVIDIA contributions, so no third party's code
is relicensed without consent.

License assignment (per-file SPDX tag on every source file):
- Apache-2.0: 515 files authored solely by NVIDIA (incl. the new Rubin/BF16
  grouped GEMM kernels and gemm_proj_rope_mxfp8 variants).
- MIT (67): files with surviving lines from external contributors in the
  issue NVIDIA#431 sign-off list, determined by git blame on develop.
- MIT (29): FlashAttention/QuACK-derived files carrying external authors'
  copyright; pinned to MIT so the Apache-2.0 root does not absorb them.

New since the previous revision of this PR (develop 3a9ed3f -> 4f75b1f):
- 26 new files headered; 63 modified files re-analyzed.
- Two new external contributors, adding 4 files to the MIT set:
  DrDirk (NVIDIA#423) -> conv_dgrad.h, conv_fprop.h, conv_wgrad.h
  Hongxiao (NVIDIA#410) -> test/python/fe_api/dsa/test_DSA_indexer_top_k.py
  Both must be added to the issue NVIDIA#431 consent list.
- rmhaskar (NVIDIA#432) verified NVIDIA-affiliated; Chase Block now commits from an
  NVIDIA address. Neither adds MIT files.
- PR NVIDIA#434 removed the Jerry Chen personal copyright from bwd_barriers.py;
  6 other DSA files still carry it and remain Category 2.

Licensing files: LICENSE.txt (Apache-2.0), LICENSE-MIT.txt, LICENSING.md
(dual-license manifest with per-file introducing-commit links),
THIRD_PARTY_LICENSES.txt, NOTICE, pyproject ('Apache-2.0 AND MIT'), README.

Every change is comment/header-only; all Python compiles.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@jiayus-nvidia
jiayus-nvidia deleted the remove-jerry-chen-copyright-bwd-barriers branch September 4, 2026 08:41
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.

3 participants