Skip to content

Conversation

jon-tow
Copy link
Contributor

@jon-tow jon-tow commented Oct 3, 2019

Adds torch::nn::CosineEmbeddingLoss module and functional support for the C++ API.

Issue: #25883

Reviewer: @yf225

@pytorchbot pytorchbot added the module: cpp Related to C++ API label Oct 3, 2019
@jerryzh168 jerryzh168 added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Oct 5, 2019
@yf225
Copy link
Contributor

yf225 commented Oct 7, 2019

@pytorchbot rebase this please

Copy link
Contributor

@yf225 yf225 left a comment

Choose a reason for hiding this comment

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

@jon-tow Thanks so much for the impeccable contribution! I will land it today after CI passes :D

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@yf225 is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@yf225 merged this pull request in 3b5d40c.

@jon-tow jon-tow deleted the c++-api/cosine-embedding branch October 8, 2019 22:30
thiagocrepaldi pushed a commit to thiagocrepaldi/pytorch that referenced this pull request Feb 4, 2020
Summary:
Adds `torch::nn::CosineEmbeddingLoss`  module and functional support for the C++ API.

Issue: pytorch#25883

Reviewer: yf225
Pull Request resolved: pytorch#27345

Differential Revision: D17801402

Pulled By: yf225

fbshipit-source-id: 0eabe80d7d36397e6667b331c3fa2f56d7a15962
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merged module: cpp Related to C++ API open source triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants