Skip to content
This repository was archived by the owner on Jul 1, 2023. It is now read-only.

Conform Tensor to ElementaryFunctions. #200

Merged
merged 2 commits into from
Jun 14, 2019

Conversation

rxwei
Copy link
Contributor

@rxwei rxwei commented Jun 11, 2019

No description provided.

@rxwei rxwei added the enhancement New feature or request label Jun 11, 2019
@rxwei
Copy link
Contributor Author

rxwei commented Jun 11, 2019

This is also semi-blocked on swiftlang/swift#25316.

@dan-zheng dan-zheng force-pushed the elementary-functions branch 2 times, most recently from b882d0e to 4612b60 Compare June 14, 2019 03:36
Conditionally conform `Tensor` to `ElementaryFunctions` where
`Scalar: TensorFlowFloatingPoint`.
@dan-zheng dan-zheng marked this pull request as ready for review June 14, 2019 04:03
@dan-zheng dan-zheng force-pushed the elementary-functions branch from 4612b60 to 51fe3e9 Compare June 14, 2019 04:03
@dan-zheng
Copy link
Member

Ready for review. cc @rxwei

// version for `FloatingPoint`.
// This is a manual definition.
@inlinable
func pow<T: BinaryFloatingPoint>(_ x: T, _ y: T) -> T {
Copy link
Member

Choose a reason for hiding this comment

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

This function is no longer necessary.

@rxwei
Copy link
Contributor Author

rxwei commented Jun 14, 2019

Will look soon!

@dan-zheng dan-zheng force-pushed the elementary-functions branch from 0c1ff41 to 8cf1a66 Compare June 14, 2019 20:01
Copy link
Contributor Author

@rxwei rxwei left a comment

Choose a reason for hiding this comment

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

LGTM

@dan-zheng dan-zheng merged commit 95e5a6d into tensorflow:master Jun 14, 2019
dan12411 pushed a commit to dan12411/swift-apis that referenced this pull request Jun 15, 2019
Conditionally conform `Tensor` to `ElementaryFunctions` where
`Scalar: TensorFlowFloatingPoint`.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request kokoro:run
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants