[Compat Test] Add cross-framework scatter_reduce test#71
Open
youge325 wants to merge 9 commits into
Open
Conversation
d612a67 to
24e3807
Compare
f39046c to
b215101
Compare
[Compat Test] Add cross-framework scatter_reduce test Add ScatterReduceTest with 16 test cases covering: - Shapes: small/medium/large 2D, empty tensor - Dtypes: Float, Double, Int, Long - Reduce modes: sum, amax, amin, mean, prod - Special cases: dim=1, include_self=false, inplace, invalid reduce - Workaround for torch replace mode segfault Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> EOF )
…inplace prod Add cross-framework comparison tests: - ScatterReduceProdFloatNoIncludeSelf - ScatterReduceInplaceProdFloatSmall Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…lf=false Add cross-framework comparison tests: - ScatterReduceAmaxFloatNoIncludeSelf - ScatterReduceAminFloatNoIncludeSelf - ScatterReduceMeanFloatNoIncludeSelf Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Change self from 5.0f to 25.0f so some src values are below self and some are above, ensuring the test can distinguish include_self=true vs false behavior. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Add ScatterReduceDimOutOfRange cross-framework comparison test for int64_t dim exceeding int range. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
b215101 to
6de8da9
Compare
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
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
Add
ScatterReduceTestfor cross-framework compatibility testing ofTensor::scatter_reduce.Test coverage (16 cases):
Test plan
result_cmp.shachieves MATCH for ScatterReduceTestRelated: PaddlePaddle/Paddle#79196
🤖 Generated with Claude Code