Skip to content

[0-size Tensor Job2 No.62] Add 0-size Tensor support for paddle.nn.functional.margin_cross_entropy - #354

Merged
cangtianhuang merged 7 commits into
PFCCLab:mainfrom
co63oc:fix6
Jul 21, 2025
Merged

cangtianhuang merged 7 commits into
PFCCLab:mainfrom
co63oc:fix6

Conversation

@co63oc

@co63oc co63oc commented Jul 8, 2025 •

Copy link
Copy Markdown
Contributor

62 paddle.nn.functional.margin_cross_entropy
PaddleAPITest 为numpy error
image

查看文档,参数2为类别,类别可能需要大于0,为0没有实际含义
https://www.paddlepaddle.org.cn/documentation/docs/zh/develop/api/paddle/nn/functional/margin_cross_entropy_cn.html#margin-cross-entropy
image

取消配置信息

166 paddle.nn.functional.ctc_loss 取消配置信息
257 paddle.signal.istft 没有配置信息没有修改
151 paddle.nn.functional.affine_grid 用例为numpy error, 取消配置信息

@paddle-bot

paddle-bot Bot commented Jul 8, 2025

Copy link
Copy Markdown

Thanks for your contribution!

@wanghuancoder wanghuancoder left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. 运行Torch报错吗?
  2. 如果要删除,不仅要删除report里的配置还需要删除tester/api_config/7_0_size里的配置

@wanghuancoder

Copy link
Copy Markdown
Collaborator
  1. 运行Torch报错吗?
  2. 如果要删除,不仅要删除report里的配置还需要删除tester/api_config/7_0_size里的配置

尽量与Torch保持一致,另外可以让Paddle报错提示0非法

@co63oc

co63oc commented Jul 8, 2025 •

Copy link
Copy Markdown
Contributor Author
  1. 运行Torch报错吗?
  2. 如果要删除,不仅要删除report里的配置还需要删除tester/api_config/7_0_size里的配置

尽量与Torch保持一致,另外可以让Paddle报错提示0非法

这里是numpy error,PaddleAPITest 生成数据的规则拦截,没有到torch运行的步骤
已删除 tester/api_config/7_0_size 的配置

class MarginCrossEntropyRule(BaseRule):

PaddleAPITest 中是自定义规则,用torch.nn.functional.softmax + torch.nn.functional.cross_entropy,没有torch对应api

@co63oc

co63oc commented Jul 13, 2025

Copy link
Copy Markdown
Contributor Author

Paddle报错的PR PaddlePaddle/Paddle#74010

@co63oc

co63oc commented Jul 18, 2025

Copy link
Copy Markdown
Contributor Author

@cangtianhuang 已在paddle提交PR修改报错,这里需要删除配置

PaddlePaddle/Paddle#74042 (comment)
image

@cangtianhuang cangtianhuang left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants