Skip to content

fix bce_with_logits with legacy reduce#12689

Closed
ailzhang wants to merge 1 commit intopytorch:masterfrom
ailzhang:fix_bce_logits
Closed

fix bce_with_logits with legacy reduce#12689
ailzhang wants to merge 1 commit intopytorch:masterfrom
ailzhang:fix_bce_logits

Conversation

@ailzhang
Copy link
Copy Markdown
Contributor

Fix #12624 . internal usecase of legacy reduce.
Add test in test_nn

Copy link
Copy Markdown
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.

ailzhang has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Copy Markdown
Collaborator

@ssnl ssnl left a comment

Choose a reason for hiding this comment

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

Let's get this in! :)

But we should also add legacy enum tests to all our loss functions.

Comment thread test/test_nn.py
)


def bce_with_logistic_legacy_enum_test():

This comment was marked as off-topic.

Copy link
Copy Markdown
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.

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

@ezyang ezyang added the merged label Jun 25, 2019
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Fix pytorch#12624 . internal usecase of legacy `reduce`.
Add test in test_nn
Pull Request resolved: pytorch#12689

Reviewed By: ezyang

Differential Revision: D10391195

Pulled By: ailzhang

fbshipit-source-id: 1af2b258c4abb2b6527eaaeac63e8bf1762c66a1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants