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

Jump to bottom

[SR-9632] [AD] Provide AD support for sum/mean #52078

Closed
swift-ci opened this issue Jan 10, 2019 · 1 comment
Closed

[SR-9632] [AD] Provide AD support for sum/mean #52078

swift-ci opened this issue Jan 10, 2019 · 1 comment
Assignees
@dan-zheng
Labels
bug A deviation from expected or documented behavior. Also: expected but undesirable behavior. swift for tensorflow

Comments

@swift-ci
Copy link
Collaborator

Previous ID SR-9632
Radar None
Original Reporter xiejw (JIRA User)
Type Bug
Status Resolved
Resolution Done
Additional Detail from JIRA
Votes 0
Component/s Swift for TensorFlow
Labels Bug
Assignee @dan-zheng
Priority Medium

md5: eeb170d0cdebe3d825a758be87ac3dc8

Issue Description:

Short context: sum and mean are common Tensorflow fns to generate the final scalar loss. So, we should have them.
@rxwei
Copy link
Member

rxwei commented Jan 10, 2019

Absolutely. We did not make this function `@differentiable` because of some complications in the generics system when it interacted with the scalar return type. The bug has been fixed, and the reduction operators have been made return a `Tensor<Scalar>`.

@swift-ci swift-ci transferred this issue from apple/swift-issues Apr 25, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dan-zheng dan-zheng

Labels
bug A deviation from expected or documented behavior. Also: expected but undesirable behavior. swift for tensorflow
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rxwei @swift-ci