Add value check for chunk

[Liyulingyue]

PR Category

User Experience

PR Types

Others

Description

#65043

[Copilot]

Pull Request Overview

This PR adds validation for the chunks parameter in paddle.chunk and corresponding error-case tests.

• Enforces that chunks must be >0 and ≤ the size of the specified (non-negative) axis.
• Raises ValueError when chunks is out of range.
• Adds unit tests for negative and too-large chunks values.

Reviewed Changes

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

File	Description
test/legacy_test/test_chunk_op.py	Added tests for chunks=-1 and chunks>dim to ensure errors.
python/paddle/tensor/manipulation.py	Inserted a guard to validate chunks bounds and raise ValueError.

Comments suppressed due to low confidence (2)

test/legacy_test/test_chunk_op.py:63

• Add a test case for a negative axis (e.g., axis=-1) with chunks larger than that dimension to verify the validation logic for negative axes.

 def test_negative_chunks_tensor(): 

python/paddle/tensor/manipulation.py:4487

• The current check only covers non-negative axes. Negative axis values (e.g., -1) bypass this validation and may lead to inconsistent behavior; consider normalizing negative axes before comparing against x.shape.

 isinstance(axis, int) and axis >= 0 and chunks > x.shape[axis] 

[Copilot]

Minor spelling issue: change 'meanless' to 'meaningless' in the comment.

Suggested change

	# check the chunks value to avoid the meanless split operation, such as <=0 or > x.shape[axis]
	# check the chunks value to avoid the meaningless split operation, such as <=0 or > x.shape[axis]

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (develop@1b1ad5a). Learn more about missing BASE report.

Additional details and impacted files

@@ Coverage Diff @@ ## develop #73857 +/- ## =========================================== Coverage ? 100.00% =========================================== Files ? 1 Lines ? 2 Branches ? 0 =========================================== Hits ? 2 Misses ? 0 Partials ? 0 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[paddle-bot]

你的PR提交成功，感谢你对开源项目的贡献!
请关注后续CI自动化测试结果，详情请参考Paddle-CI手册。
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.