Thanks to visit codestin.com
Credit goes to github.com

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

Conversation

efremale
Copy link
Contributor

Since TF-433 was fixed, we can remove all the workarkounds for (non-active) try-apply.
For example, calls to Array.map(_:).

Since TF-433 was fixed, we can remove all the workarkounds for (non-active) try-apply.
For example, calls to `Array.map(_:)`.
@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@efremale
Copy link
Contributor Author

@googlebot I signed it!

@googlebot
Copy link

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

Copy link
Member

@dan-zheng dan-zheng left a comment

Choose a reason for hiding this comment

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

Nice!

@dan-zheng
Copy link
Member

Tests fail because the toolchains used for running CI haven't been updated in a while due to build failures. Once the build failures are fixed and toolchains are updated, CI should pass.

@dan-zheng
Copy link
Member

dan-zheng commented Sep 10, 2020

Test pass now that the toolchains used for running CI have been updated.

This PR depends on Swift compiler changes after the Swift for TensorFlow 0.11 release, so let's wait until the next release to merge this PR so that tensorflow/swift-apis continues to build with the latest release.

@dan-zheng
Copy link
Member

Verifying that tests pass one more time.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants