docs(legal): PinchBench modified-component notice and NVIDIA disclaimer - #2629
Merged
Conversation
…laimer - Add "Cloned and Modified Components" section to ATTRIBUTIONS.md documenting that PinchBench skill is cloned at image build time and patched via nvidia-pinchbench.patch (modifies scripts/lib_agent.py for NVIDIA endpoint integration); includes MIT license text - Add NVIDIA retrieved-materials Notice and Disclaimer to README.md as required for containers that automatically retrieve external materials (models, data, assets) at runtime Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Kajal Jain <kajalj@nvidia.com>
Contributor
Author
|
/ok to test 20ebc55 |
kajalj22
enabled auto-merge (squash)
August 20, 2026 01:57
Signed-off-by: Kajal Jain <kajalj@nvidia.com>
… row only Signed-off-by: Kajal Jain <kajalj@nvidia.com>
chtruong814
approved these changes
Aug 20, 2026
Contributor
Author
|
/ok to test 7103778 |
This was referenced Sep 2, 2026
Merged
kajalj22
added a commit
that referenced
this pull request
Sep 2, 2026
…er (#2629) - Adds a **"Cloned and Modified Components"** section to `ATTRIBUTIONS.md` covering PinchBench: cloned at image build time from https://github.com/pinchbench/skill and patched via `nvidia-pinchbench.patch` (touches `scripts/lib_agent.py` only). Includes MIT license text. Addresses Legal's finding that PinchBench being "cloned & patched at image build" is a modification requiring explicit attribution. - Adds the **NVIDIA retrieved-materials Notice and Disclaimer** to `README.md` as required for containers that automatically retrieve external materials (models, data, assets) at runtime. Closes items 10 and 11 from OSRB guidance. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Signed-off-by: Kajal Jain <kajalj@nvidia.com> Co-authored-by: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Kajal Jain <kajalj@nvidia.com>
kajalj22
added a commit
that referenced
this pull request
Sep 2, 2026
…er (#2629) - Adds a **"Cloned and Modified Components"** section to `ATTRIBUTIONS.md` covering PinchBench: cloned at image build time from https://github.com/pinchbench/skill and patched via `nvidia-pinchbench.patch` (touches `scripts/lib_agent.py` only). Includes MIT license text. Addresses Legal's finding that PinchBench being "cloned & patched at image build" is a modification requiring explicit attribution. - Adds the **NVIDIA retrieved-materials Notice and Disclaimer** to `README.md` as required for containers that automatically retrieve external materials (models, data, assets) at runtime. Closes items 10 and 11 from OSRB guidance. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Signed-off-by: Kajal Jain <kajalj@nvidia.com> Co-authored-by: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Kajal Jain <kajalj@nvidia.com>
kajalj22
added a commit
that referenced
this pull request
Sep 2, 2026
…laimer (#2629) into main (#2985) ## Summary - Cherry-pick of #2629 from `r0.5.1` into `main`. **Stack position 2 of 5** — #2547 (position 1) merged as #2986. This is now next in line, rebased directly onto `main`. - Adds a "Cloned and Modified Components" section to `ATTRIBUTIONS.md` covering PinchBench (cloned & patched at image build, MIT-licensed). - Adds the NVIDIA retrieved-materials Notice and Disclaimer to `README.md`, required for containers that automatically retrieve external materials at runtime. - Addresses OSRB guidance items 10 and 11. Remaining stack (merge in this order): #2985 → #2984 → #2987 → #2983 ## Test plan - [ ] Confirm ATTRIBUTIONS.md and README.md render correctly Signed-off-by: Kajal Jain <kajalj@nvidia.com> Co-authored-by: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
kajalj22
added a commit
that referenced
this pull request
Sep 4, 2026
#2725) into main (#2984) ## Summary - Cherry-pick of #2725 from `r0.5.1` into `main`. **Stack position 3 of 5** — #2547 and #2629 (positions 1-2) merged as #2986 and #2985. This is now next in line, rebased directly onto `main`. - Updates installation docs: Python 3.12 → 3.13.14 references, adds the pre-built NeMo Gym NGC container instructions (replacing the outdated NeMo RL v0.4.0 container guidance), and adds the `nvidia-container-toolkit` install step + docs for the enroot GPU hook. Signed-off-by: Kajal Jain <kajalj@nvidia.com>
2 tasks
kajalj22
added a commit
that referenced
this pull request
Sep 4, 2026
…2987) ## Summary - Cherry-pick of #2731 from `r0.5.1` into `main`. **Stack position 4 of 5** — #2547, #2629, #2725 (positions 1-3) merged as #2986, #2985, #2984. This is now next in line, rebased directly onto `main`. - Adds the frozen `fern/versions/v0.5.1/` docs snapshot (and `fern/versions/v0.5.1.yml`) for the v0.5.1 GA release, mirroring how prior GA versions (v0.5.0, v0.4.0, ...) got their own version snapshot in the docs site. - Also inserts the v0.5.1 accordion into `release-notes.mdx`, between the newer v0.6.0 entry (already on `main`) and v0.5.0. - Note on PR title: intentionally omits `(#2731)` — the release-branch auto-cherrypick automation (`cherry-pick-release-commit.yml`) naively grabs the *first* `(#NNNN)` in the squash-merge commit subject to resolve "which PR is this". Embedding the original PR number in the title broke that lookup for #2984/#2985/#2986 (see #3091/#3092 for the manual r0.6.0 backfills). Keeping this title number-free lets the automation resolve the real PR (#2987) and auto-create the r0.6.0 cherry-pick. Remaining stack (merge in this order): #2987 → #2983 ## Test plan - [ ] Confirm fern docs build picks up the new v0.5.1 version and the version switcher shows it - [ ] `tests/unit_tests/test_fern_docs_links.py` passes Signed-off-by: Kajal Jain <kajalj@nvidia.com>
kajalj22
added a commit
that referenced
this pull request
Sep 5, 2026
…laimer (#2629) into main (#2985) ## Summary - Cherry-pick of #2629 from `r0.5.1` into `main`. **Stack position 2 of 5** — #2547 (position 1) merged as #2986. This is now next in line, rebased directly onto `main`. - Adds a "Cloned and Modified Components" section to `ATTRIBUTIONS.md` covering PinchBench (cloned & patched at image build, MIT-licensed). - Adds the NVIDIA retrieved-materials Notice and Disclaimer to `README.md`, required for containers that automatically retrieve external materials at runtime. - Addresses OSRB guidance items 10 and 11. Remaining stack (merge in this order): #2985 → #2984 → #2987 → #2983 ## Test plan - [ ] Confirm ATTRIBUTIONS.md and README.md render correctly Signed-off-by: Kajal Jain <kajalj@nvidia.com> Co-authored-by: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Kajal Jain <kajalj@nvidia.com>
kajalj22
added a commit
that referenced
this pull request
Sep 5, 2026
…claimer (2629)` into `r0.6.0` (#3093) ## Summary - Cherry-pick of #2629 into `r0.6.0` - Adds "Cloned and Modified Components" section to `ATTRIBUTIONS.md` (PinchBench) - Adds NVIDIA Notice and Disclaimer to `README.md` Signed-off-by: Kajal Jain <kajalj@nvidia.com> Co-authored-by: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
jkyi-nvidia
pushed a commit
that referenced
this pull request
Sep 11, 2026
…laimer (#2629) into main (#2985) ## Summary - Cherry-pick of #2629 from `r0.5.1` into `main`. **Stack position 2 of 5** — #2547 (position 1) merged as #2986. This is now next in line, rebased directly onto `main`. - Adds a "Cloned and Modified Components" section to `ATTRIBUTIONS.md` covering PinchBench (cloned & patched at image build, MIT-licensed). - Adds the NVIDIA retrieved-materials Notice and Disclaimer to `README.md`, required for containers that automatically retrieve external materials at runtime. - Addresses OSRB guidance items 10 and 11. Remaining stack (merge in this order): #2985 → #2984 → #2987 → #2983 ## Test plan - [ ] Confirm ATTRIBUTIONS.md and README.md render correctly Signed-off-by: Kajal Jain <kajalj@nvidia.com> Co-authored-by: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Junkeun Yi <jkyi@nvidia.com>
jkyi-nvidia
pushed a commit
that referenced
this pull request
Sep 11, 2026
#2725) into main (#2984) ## Summary - Cherry-pick of #2725 from `r0.5.1` into `main`. **Stack position 3 of 5** — #2547 and #2629 (positions 1-2) merged as #2986 and #2985. This is now next in line, rebased directly onto `main`. - Updates installation docs: Python 3.12 → 3.13.14 references, adds the pre-built NeMo Gym NGC container instructions (replacing the outdated NeMo RL v0.4.0 container guidance), and adds the `nvidia-container-toolkit` install step + docs for the enroot GPU hook. Signed-off-by: Kajal Jain <kajalj@nvidia.com> Signed-off-by: Junkeun Yi <jkyi@nvidia.com>
jkyi-nvidia
pushed a commit
that referenced
this pull request
Sep 11, 2026
…2987) ## Summary - Cherry-pick of #2731 from `r0.5.1` into `main`. **Stack position 4 of 5** — #2547, #2629, #2725 (positions 1-3) merged as #2986, #2985, #2984. This is now next in line, rebased directly onto `main`. - Adds the frozen `fern/versions/v0.5.1/` docs snapshot (and `fern/versions/v0.5.1.yml`) for the v0.5.1 GA release, mirroring how prior GA versions (v0.5.0, v0.4.0, ...) got their own version snapshot in the docs site. - Also inserts the v0.5.1 accordion into `release-notes.mdx`, between the newer v0.6.0 entry (already on `main`) and v0.5.0. - Note on PR title: intentionally omits `(#2731)` — the release-branch auto-cherrypick automation (`cherry-pick-release-commit.yml`) naively grabs the *first* `(#NNNN)` in the squash-merge commit subject to resolve "which PR is this". Embedding the original PR number in the title broke that lookup for #2984/#2985/#2986 (see #3091/#3092 for the manual r0.6.0 backfills). Keeping this title number-free lets the automation resolve the real PR (#2987) and auto-create the r0.6.0 cherry-pick. Remaining stack (merge in this order): #2987 → #2983 ## Test plan - [ ] Confirm fern docs build picks up the new v0.5.1 version and the version switcher shows it - [ ] `tests/unit_tests/test_fern_docs_links.py` passes Signed-off-by: Kajal Jain <kajalj@nvidia.com> Signed-off-by: Junkeun Yi <jkyi@nvidia.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ATTRIBUTIONS.mdcovering PinchBench: cloned at image build time from https://github.com/pinchbench/skill and patched vianvidia-pinchbench.patch(touchesscripts/lib_agent.pyonly). Includes MIT license text. Addresses Legal's finding that PinchBench being "cloned & patched at image build" is a modification requiring explicit attribution.README.mdas required for containers that automatically retrieve external materials (models, data, assets) at runtime.Closes items 10 and 11 from OSRB guidance.
🤖 Generated with Claude Code