-
Notifications
You must be signed in to change notification settings - Fork 137
Generate Raw ops from Tensorflow2.1.0rc1 #621
Conversation
@dan-zheng is build time on kokoro really on the order of 30 minutes and more? |
Could be. CI tests more than just tensorflow/swift-apis, also tensorflow/swift-models and other related repositories to ensure they're not broken. |
@dan-zheng Changed the error bound and checked locally, the tests pass. i think this can be merged now We should probably also change the commit hash in the tensorflow branch, since this is an api update? |
Yep, feel free to submit a PR to apple/swift. Here's an example: swiftlang/swift#28934 You'll want to wait until this PR is merged, though, to use the squashed+merged commit hash. |
Fixes: #619
Execute has to be overloaded up to 14 variables due to the existence of
parseSequenceExampleV2
.Updated
generatewrappers.swift
TensorFlow import command.High-arity return statement functions in
RawOpsGenerated.swift
have to be broken down by using let variables.The accuracy of svd had to be changed for the tests to pass.
Build and test passes.