Bump copyright end-year to 2026 in 153 files#899
Open
orbalayla-nvidia wants to merge 1 commit into
Open
Conversation
The CloudAI release pipeline's copyright checker requires the SPDX
header's copyright line to extend through the current year. These 153
.py / .toml / Makefile files were stuck on 2024-2025 / 2025 / 2024 /
'2024, 2026' and were the only thing failing the Check copyrights
stage. Brings them to {start}-2026 (and normalizes the one comma-list
file to a range).
Signed-off-by: Or Balayla <obalayla@nvidia.com>
orbalayla-nvidia
requested review from
jeffnvidia,
podkidyshev and
srivatsankrishnan
as code owners
Contributor
📝 WalkthroughWalkthroughThis pull request updates copyright year ranges across the entire repository from 2024-2025/2025 to 2024-2026/2025-2026. Changes span configuration files (TOML), Python source code, and test files, with standardized "NVIDIA CORPORATION & AFFILIATES" attribution where needed. ChangesCopyright Year Update to 2026
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 11
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@conf/common/system/standalone_system.toml`:
- Line 2: Replace the year range in the copyright header string "# Copyright (c)
2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved." with a
comma-separated list for non-consecutive years so it reads "# Copyright (c)
2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved."; update the
exact header text found in the standalone_system.toml content to use "2024,
2026" instead of "2024-2026".
In `@conf/release/spcx/l40s/test_scenario/l40s_bc_nccl_test.toml`:
- Line 2: Replace the hyphenated year range in the copyright header with a
comma-separated list: change the line that reads "# Copyright (c) 2024-2026
NVIDIA CORPORATION & AFFILIATES. All rights reserved." to use a comma between
years (e.g., "# Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All
rights reserved.") so the checker accepts the format.
In `@conf/release/spcx/l40s/test/l40s-bc-nccl_test_scatter.toml`:
- Line 2: Replace the year range format "2024-2026" in the file header string
with comma-separated years so the copyright line reads "Copyright (c) 2024, 2026
NVIDIA CORPORATION & AFFILIATES. All rights reserved."; specifically update the
existing header comment that currently contains "# Copyright (c) 2024-2026
NVIDIA CORPORATION & AFFILIATES. All rights reserved." to use the
comma-separated form.
In `@conf/release/spcx/l40s/test/l40s-bc-nccl_test_sendrecv_worst.toml`:
- Line 2: Update the copyright year format to use comma-separated years instead
of a hyphen; specifically replace the line currently containing "Copyright (c)
2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved." with "Copyright
(c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved." so it
matches the copyright checker expectation.
In `@conf/release/spcx/l40s/test/l40s-bc-nccl_test_sendrecv.toml`:
- Line 2: Update the copyright header to use comma-separated years rather than a
hyphen: replace the existing line that reads "# Copyright (c) 2024-2026 NVIDIA
CORPORATION & AFFILIATES. All rights reserved." with "# Copyright (c) 2024, 2026
NVIDIA CORPORATION & AFFILIATES. All rights reserved." so the copyright checker
accepts the format.
In `@doc/Makefile`:
- Line 2: Update the copyright header line that currently reads "Copyright (c)
2025-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved." to use
comma-separated years instead of a hyphen; change the year range to "2025, 2026"
so the line becomes "Copyright (c) 2025, 2026 NVIDIA CORPORATION & AFFILIATES.
All rights reserved." to match the other files and satisfy the copyright
checker.
In `@src/cloudai/_core/__init__.py`:
- Line 2: Update the copyright header string that currently reads "Copyright (c)
2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved." to use
comma-separated years instead of a hyphen; change it to "Copyright (c) 2024,
2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved." ensuring the exact
header line in src/cloudai/_core/__init__.py (the copyright header) is replaced
accordingly so it matches the CI checker.
In `@src/cloudai/_core/base_job.py`:
- Line 2: Replace the copyright year range string that uses a hyphen
("2024-2026") with a comma-separated list ("2024, 2026") in the file header;
update the existing header comment line that begins with "# Copyright (c)" so it
reads "# Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights
reserved." to satisfy the copyright checker.
In `@src/cloudai/_core/base_system_parser.py`:
- Line 2: Update the copyright year format in
src/cloudai/_core/base_system_parser.py by replacing the range "2024-2026" with
the comma-separated years "2024, 2026" so it matches the pipeline's expected
format; locate the file header line in base_system_parser.py and modify the
copyright string accordingly (ensure other nearby files follow the same "YYYY,
YYYY" pattern if present).
In `@src/cloudai/report_generator/tool/__init__.py`:
- Line 2: Update the copyright header in this module to use comma-separated
years instead of a dash: replace the line in
src/cloudai/report_generator/tool/__init__.py that currently reads "# Copyright
(c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved." with "#
Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.";
do the same change for any other modified files that list non-contiguous years
(e.g., other __init__.py or module headers) so CI's "Check copyrights" passes.
In `@src/cloudai/util/command_shell.py`:
- Line 2: Update the copyright header string that currently reads "2024-2026
NVIDIA CORPORATION & AFFILIATES. All rights reserved." to use a comma-separated,
non-contiguous year list ("2024, 2026 NVIDIA CORPORATION & AFFILIATES. All
rights reserved.") so it matches the repository's copyright checker; locate and
edit the top-of-file copyright comment in src/cloudai/util/command_shell.py (the
header string) accordingly.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Enterprise
Run ID: d55ddd85-367a-4261-a6aa-8205357e2e1d
📒 Files selected for processing (153)
conf/common/system/example_runai_cluster.tomlconf/common/system/example_slurm_cluster.tomlconf/common/system/kubernetes_cluster.tomlconf/common/system/standalone_system.tomlconf/common/test_scenario/dse_nccl_all_gather.tomlconf/common/test_scenario/nemo_run_llama3_8b.tomlconf/common/test_scenario/sleep.tomlconf/common/test_scenario/ucc_test.tomlconf/experimental/aiconfigurator/test/dse_aiconfigurator_disagg.tomlconf/experimental/test/ddlb_test.tomlconf/experimental/test/nemo_launcher_nemotron_15b_bf16_128_node.tomlconf/experimental/test/nemo_launcher_nemotron_15b_bf16_256_node.tomlconf/experimental/test/nemo_launcher_nemotron_15b_bf16_2_node.tomlconf/experimental/test/nemo_launcher_nemotron_15b_fp8_128_node.tomlconf/experimental/test/nemo_launcher_nemotron_15b_fp8_16_node.tomlconf/experimental/test/nemo_launcher_nemotron_15b_fp8_256_node.tomlconf/experimental/test/nemo_launcher_nemotron_15b_fp8_2_node.tomlconf/experimental/test/nemo_launcher_nemotron_15b_fp8_32_node.tomlconf/experimental/test/nemo_launcher_nemotron_15b_fp8_4_node.tomlconf/experimental/test/nemo_launcher_nemotron_15b_fp8_64_node.tomlconf/experimental/test/nemo_launcher_nemotron_15b_fp8_8_node.tomlconf/experimental/test_scenario/nemo_launcher_nemotron_15b_bf16.tomlconf/experimental/test_scenario/nemo_launcher_nemotron_15b_fp8.tomlconf/hook/nccl_test.tomlconf/release/nemo_acceptance/test/gpt3_126m_mock.tomlconf/release/nemo_acceptance/test/gpt3_126m_pile.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_all_gather.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_all_gather_worst.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_all_reduce.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_all_reduce_worst.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_alltoall.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_alltoall_worst.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_alltoall_worst_failover.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_bisection.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_broadcast.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_gather.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_hypercube.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_reduce.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_reduce_scatter.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_reduce_scatter_worst.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_scatter.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_sendrecv.tomlconf/release/spcx/l40s/test/l40s-bc-nccl_test_sendrecv_worst.tomlconf/release/spcx/l40s/test_scenario/l40s_bc_nccl_test.tomldoc/Makefilesrc/cloudai/__init__.pysrc/cloudai/_core/__init__.pysrc/cloudai/_core/base_job.pysrc/cloudai/_core/base_system_parser.pysrc/cloudai/_core/exceptions.pysrc/cloudai/_core/grader.pysrc/cloudai/_core/grading_strategy.pysrc/cloudai/_core/job_status_result.pysrc/cloudai/_core/system.pysrc/cloudai/cli/__init__.pysrc/cloudai/cli/cli.pysrc/cloudai/configurator/reward_functions.pysrc/cloudai/report_generator/groups.pysrc/cloudai/report_generator/tool/__init__.pysrc/cloudai/report_generator/tool/bokeh_report_tool.pysrc/cloudai/report_generator/tool/csv_report_tool.pysrc/cloudai/report_generator/tool/report_tool_interface.pysrc/cloudai/report_generator/tool/tensorboard_data_reader.pysrc/cloudai/systems/__init__.pysrc/cloudai/systems/kubernetes/__init__.pysrc/cloudai/systems/kubernetes/kubernetes_job.pysrc/cloudai/systems/lsf/lsf_command_gen_strategy.pysrc/cloudai/systems/lsf/lsf_job.pysrc/cloudai/systems/lsf/lsf_node.pysrc/cloudai/systems/lsf/lsf_runner.pysrc/cloudai/systems/runai/runai_cluster.pysrc/cloudai/systems/runai/runai_event.pysrc/cloudai/systems/runai/runai_job.pysrc/cloudai/systems/runai/runai_node.pysrc/cloudai/systems/runai/runai_project.pysrc/cloudai/systems/runai/runai_pvc.pysrc/cloudai/systems/runai/runai_training.pysrc/cloudai/systems/slurm/__init__.pysrc/cloudai/systems/slurm/docker_image_cache_manager.pysrc/cloudai/systems/slurm/slurm_job.pysrc/cloudai/systems/slurm/slurm_metadata.pysrc/cloudai/systems/slurm/slurm_node.pysrc/cloudai/systems/standalone/standalone_job.pysrc/cloudai/systems/standalone/standalone_runner.pysrc/cloudai/systems/standalone/standalone_system.pysrc/cloudai/util/__init__.pysrc/cloudai/util/command_shell.pysrc/cloudai/util/lazy_imports.pysrc/cloudai/util/utils.pysrc/cloudai/workloads/__init__.pysrc/cloudai/workloads/aiconfig/__init__.pysrc/cloudai/workloads/bash_cmd/__init__.pysrc/cloudai/workloads/bash_cmd/bash_cmd.pysrc/cloudai/workloads/chakra_replay/chakra_replay.pysrc/cloudai/workloads/chakra_replay/grading_strategy.pysrc/cloudai/workloads/chakra_replay/report_generation_strategy.pysrc/cloudai/workloads/chakra_replay/slurm_command_gen_strategy.pysrc/cloudai/workloads/ddlb/__init__.pysrc/cloudai/workloads/ddlb/ddlb.pysrc/cloudai/workloads/ddlb/slurm_command_gen_strategy.pysrc/cloudai/workloads/jax_toolbox/gpt.pysrc/cloudai/workloads/jax_toolbox/grading_strategy.pysrc/cloudai/workloads/jax_toolbox/grok.pysrc/cloudai/workloads/jax_toolbox/jax_toolbox.pysrc/cloudai/workloads/jax_toolbox/nemotron.pysrc/cloudai/workloads/jax_toolbox/report_generation_strategy.pysrc/cloudai/workloads/jax_toolbox/slurm_command_gen_strategy.pysrc/cloudai/workloads/nccl_test/grading_strategy.pysrc/cloudai/workloads/nccl_test/nccl.pysrc/cloudai/workloads/nccl_test/nccl_comparison_report.pysrc/cloudai/workloads/nccl_test/prediction_report_generation_strategy.pysrc/cloudai/workloads/nccl_test/runai_json_gen_strategy.pysrc/cloudai/workloads/nccl_test/slurm_command_gen_strategy.pysrc/cloudai/workloads/nemo_launcher/grading_strategy.pysrc/cloudai/workloads/nemo_launcher/nemo_launcher.pysrc/cloudai/workloads/nemo_launcher/report_generation_strategy.pysrc/cloudai/workloads/nemo_launcher/slurm_command_gen_strategy.pysrc/cloudai/workloads/nemo_run/cloudai_nemorun.pysrc/cloudai/workloads/nemo_run/data_store_report_generation_strategy.pysrc/cloudai/workloads/nemo_run/http_data_repository.pysrc/cloudai/workloads/nemo_run/slurm_command_gen_strategy.pysrc/cloudai/workloads/nixl_bench/__init__.pysrc/cloudai/workloads/nixl_bench/nixl_summary_report.pysrc/cloudai/workloads/nixl_kvbench/__init__.pysrc/cloudai/workloads/nixl_perftest/__init__.pysrc/cloudai/workloads/nixl_perftest/report_generation_strategy.pysrc/cloudai/workloads/osu_bench/slurm_command_gen_strategy.pysrc/cloudai/workloads/sleep/__init__.pysrc/cloudai/workloads/sleep/grading_strategy.pysrc/cloudai/workloads/sleep/lsf_command_gen_strategy.pysrc/cloudai/workloads/sleep/sleep.pysrc/cloudai/workloads/sleep/slurm_command_gen_strategy.pysrc/cloudai/workloads/sleep/standalone_command_gen_strategy.pysrc/cloudai/workloads/slurm_container/slurm_command_gen_strategy.pysrc/cloudai/workloads/slurm_container/slurm_container.pysrc/cloudai/workloads/triton_inference/__init__.pysrc/cloudai/workloads/triton_inference/report_generation_strategy.pysrc/cloudai/workloads/triton_inference/slurm_command_gen_strategy.pysrc/cloudai/workloads/triton_inference/triton_inference.pysrc/cloudai/workloads/ucc_test/__init__.pysrc/cloudai/workloads/ucc_test/grading_strategy.pysrc/cloudai/workloads/ucc_test/slurm_command_gen_strategy.pysrc/cloudai/workloads/ucc_test/ucc.pytests/report_generation_strategy/test_checkpoint_timings.pytests/report_generation_strategy/test_data_store_report_generation_strategy.pytests/report_generation_strategy/test_report_groups.pytests/test_command_gen_strategy.pytests/test_csv_report_tool.pytests/test_docker_image_cache_manager.pytests/test_job_type_handler.pytests/test_registry.pytests/test_test_run.pytests/test_test_scenario_parser.py
| @@ -1,5 +1,5 @@ | |||
| # SPDX-FileCopyrightText: NVIDIA CORPORATION & AFFILIATES | |||
| # Copyright (c) 2024 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
Contributor
There was a problem hiding this comment.
Copyright format causes pipeline test failure.
The test suite expects # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. (comma-separated for year gaps) but this line uses # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. (range format). This mismatch is blocking the CI pipeline.
Either update the test to accept range format for gaps, or use comma-separated format here: 2024, 2026.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@conf/common/system/standalone_system.toml` at line 2, Replace the year range
in the copyright header string "# Copyright (c) 2024-2026 NVIDIA CORPORATION &
AFFILIATES. All rights reserved." with a comma-separated list for
non-consecutive years so it reads "# Copyright (c) 2024, 2026 NVIDIA CORPORATION
& AFFILIATES. All rights reserved."; update the exact header text found in the
standalone_system.toml content to use "2024, 2026" instead of "2024-2026".
| @@ -1,5 +1,5 @@ | |||
| # SPDX-FileCopyrightText: NVIDIA CORPORATION & AFFILIATES | |||
| # Copyright (c) 2024 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
Contributor
There was a problem hiding this comment.
Copyright year format must use comma separation, not hyphen.
The copyright checker expects # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. but the file uses 2024-2026. The CI pipeline is failing on this exact format mismatch.
🔧 Fix the copyright format
-# Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
+# Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |
| # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@conf/release/spcx/l40s/test_scenario/l40s_bc_nccl_test.toml` at line 2,
Replace the hyphenated year range in the copyright header with a comma-separated
list: change the line that reads "# Copyright (c) 2024-2026 NVIDIA CORPORATION &
AFFILIATES. All rights reserved." to use a comma between years (e.g., "#
Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.")
so the checker accepts the format.
| @@ -1,5 +1,5 @@ | |||
| # SPDX-FileCopyrightText: NVIDIA CORPORATION & AFFILIATES | |||
| # Copyright (c) 2024-2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
Contributor
There was a problem hiding this comment.
Copyright year format must use comma separation, not hyphen.
The copyright checker expects # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. but the file uses 2024-2026. The CI pipeline is failing on this exact format mismatch.
🔧 Fix the copyright format
-# Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
+# Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |
| # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@conf/release/spcx/l40s/test/l40s-bc-nccl_test_scatter.toml` at line 2,
Replace the year range format "2024-2026" in the file header string with
comma-separated years so the copyright line reads "Copyright (c) 2024, 2026
NVIDIA CORPORATION & AFFILIATES. All rights reserved."; specifically update the
existing header comment that currently contains "# Copyright (c) 2024-2026
NVIDIA CORPORATION & AFFILIATES. All rights reserved." to use the
comma-separated form.
| @@ -1,5 +1,5 @@ | |||
| # SPDX-FileCopyrightText: NVIDIA CORPORATION & AFFILIATES | |||
| # Copyright (c) 2024-2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
Contributor
There was a problem hiding this comment.
Copyright year format must use comma separation, not hyphen.
The copyright checker expects # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. but the file uses 2024-2026. The CI pipeline is failing on this exact format mismatch.
🔧 Fix the copyright format
-# Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
+# Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |
| # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@conf/release/spcx/l40s/test/l40s-bc-nccl_test_sendrecv_worst.toml` at line 2,
Update the copyright year format to use comma-separated years instead of a
hyphen; specifically replace the line currently containing "Copyright (c)
2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved." with "Copyright
(c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved." so it
matches the copyright checker expectation.
| @@ -1,5 +1,5 @@ | |||
| # SPDX-FileCopyrightText: NVIDIA CORPORATION & AFFILIATES | |||
| # Copyright (c) 2024-2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
Contributor
There was a problem hiding this comment.
Copyright year format must use comma separation, not hyphen.
The copyright checker expects # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. but the file uses 2024-2026. The CI pipeline is failing on this exact format mismatch.
🔧 Fix the copyright format
-# Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
+# Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |
| # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@conf/release/spcx/l40s/test/l40s-bc-nccl_test_sendrecv.toml` at line 2,
Update the copyright header to use comma-separated years rather than a hyphen:
replace the existing line that reads "# Copyright (c) 2024-2026 NVIDIA
CORPORATION & AFFILIATES. All rights reserved." with "# Copyright (c) 2024, 2026
NVIDIA CORPORATION & AFFILIATES. All rights reserved." so the copyright checker
accepts the format.
| @@ -1,5 +1,5 @@ | |||
| # SPDX-FileCopyrightText: NVIDIA CORPORATION & AFFILIATES | |||
| # Copyright (c) 2024 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
Contributor
There was a problem hiding this comment.
Copyright year format must use comma separation, not hyphen.
The copyright checker expects # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. but the file uses 2024-2026. The CI pipeline is failing on this exact format mismatch.
🔧 Fix the copyright format
-# Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
+# Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |
| # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/cloudai/_core/__init__.py` at line 2, Update the copyright header string
that currently reads "Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES.
All rights reserved." to use comma-separated years instead of a hyphen; change
it to "Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights
reserved." ensuring the exact header line in src/cloudai/_core/__init__.py (the
copyright header) is replaced accordingly so it matches the CI checker.
| @@ -1,5 +1,5 @@ | |||
| # SPDX-FileCopyrightText: NVIDIA CORPORATION & AFFILIATES | |||
| # Copyright (c) 2024 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
Contributor
There was a problem hiding this comment.
Copyright year format must use comma separation, not hyphen.
The copyright checker expects # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. but the file uses 2024-2026. The CI pipeline is failing on this exact format mismatch.
🔧 Fix the copyright format
-# Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
+# Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |
| # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/cloudai/_core/base_job.py` at line 2, Replace the copyright year range
string that uses a hyphen ("2024-2026") with a comma-separated list ("2024,
2026") in the file header; update the existing header comment line that begins
with "# Copyright (c)" so it reads "# Copyright (c) 2024, 2026 NVIDIA
CORPORATION & AFFILIATES. All rights reserved." to satisfy the copyright
checker.
| @@ -1,5 +1,5 @@ | |||
| # SPDX-FileCopyrightText: NVIDIA CORPORATION & AFFILIATES | |||
| # Copyright (c) 2024 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
Contributor
There was a problem hiding this comment.
Copyright year format fails pipeline validation.
The copyright checker expects comma-separated years 2024, 2026 but this PR uses range format 2024-2026. This mismatch is causing all pipeline tests to fail and blocks merge.
🔧 Proposed fix
-# Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
+# Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |
| # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/cloudai/_core/base_system_parser.py` at line 2, Update the copyright year
format in src/cloudai/_core/base_system_parser.py by replacing the range
"2024-2026" with the comma-separated years "2024, 2026" so it matches the
pipeline's expected format; locate the file header line in base_system_parser.py
and modify the copyright string accordingly (ensure other nearby files follow
the same "YYYY, YYYY" pattern if present).
| @@ -1,5 +1,5 @@ | |||
| # SPDX-FileCopyrightText: NVIDIA CORPORATION & AFFILIATES | |||
| # Copyright (c) 2024 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
Contributor
There was a problem hiding this comment.
Copyright year format must use comma, not dash, for non-contiguous years.
The CI test expects # Copyright (c) 2024, 2026 but this PR uses # Copyright (c) 2024-2026. The dash notation implies a continuous range (all years from 2024 through 2026), while the comma notation indicates only the listed years (2024 and 2026, skipping 2025). The test failure blocks the PR's stated objective of passing the "Check copyrights" stage.
Apply this format correction to all affected files where work occurred in 2024 and 2026 but not 2025:
🔧 Suggested format correction
-# Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
+# Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |
| # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/cloudai/report_generator/tool/__init__.py` at line 2, Update the
copyright header in this module to use comma-separated years instead of a dash:
replace the line in src/cloudai/report_generator/tool/__init__.py that currently
reads "# Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights
reserved." with "# Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All
rights reserved."; do the same change for any other modified files that list
non-contiguous years (e.g., other __init__.py or module headers) so CI's "Check
copyrights" passes.
| @@ -1,5 +1,5 @@ | |||
| # SPDX-FileCopyrightText: NVIDIA CORPORATION & AFFILIATES | |||
| # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
Contributor
There was a problem hiding this comment.
Copyright format mismatch causes CI failure.
The copyright year format 2024-2026 implies a continuous range, but this file's history spans non-contiguous years (2024, 2026). The repository's copyright checker test expects comma-separated format 2024, 2026 for non-contiguous years, as evidenced by the CI failures.
🔧 Proposed fix
-# Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
+# Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| # Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |
| # Copyright (c) 2024, 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/cloudai/util/command_shell.py` at line 2, Update the copyright header
string that currently reads "2024-2026 NVIDIA CORPORATION & AFFILIATES. All
rights reserved." to use a comma-separated, non-contiguous year list ("2024,
2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.") so it matches the
repository's copyright checker; locate and edit the top-of-file copyright
comment in src/cloudai/util/command_shell.py (the header string) accordingly.
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
1 participant
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
The SPDX header's copyright line should extend through the current year.
This PR brings them all to
{start}-2026(and normalizes the one comma-list file to a proper range). No content changes outside the copyright line.