Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

映射文档 No.57 torch.nn.MultiLabelMarginLoss #6058

Merged

Conversation

RedContritio
Copy link
Contributor

@RedContritio RedContritio commented Jul 25, 2023

torch.nn.MultiLabelMarginLoss 特点:

  • 含有历史遗留参数,映射时需要按照逻辑进行参数的覆盖。
  • 映射到 paddle.nn.MultiLabelSoftMarginLoss (不是一一映射,但是行为上是能直接映射的)。

@zhwesky2010
Copy link
Collaborator

@RedContritio 这个写了Matcher转换规则没?对比了结果一致吗,因为这两个名字不同,不太确定是否是对应一致的功能

@RedContritio
Copy link
Contributor Author

@RedContritio 这个写了Matcher转换规则没?对比了结果一致吗,因为这两个名字不同,不太确定是否是对应一致的功能

还没有写,按照流程是先写映射文档的。
之所以判断功能一样,是根据这两个api文档页面的公式对比判断的,尚未经过测试。

Copy link
Collaborator

@zhwesky2010 zhwesky2010 left a comment

Choose a reason for hiding this comment

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

好那先假设是对应一致吧,后面写Matcher如果又发现了问题,需要再及时更正文档的问题

@zhwesky2010 zhwesky2010 merged commit 98b7b39 into PaddlePaddle:develop Aug 3, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants