Skip to content

Upgrade TensorFlow to 2.3.1 #122

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

Merged
merged 1 commit into from
Sep 30, 2020

Conversation

saudet
Copy link
Contributor

@saudet saudet commented Sep 30, 2020

Fixes #120

@karllessard
Copy link
Collaborator

While this is building, double-checking with you @saudet , did any new generated classes appeared in your source tree after you've build this locally?

@karllessard karllessard merged commit cc5f9ad into tensorflow:master Sep 30, 2020
@saudet
Copy link
Contributor Author

saudet commented Sep 30, 2020

No, only the fields in Ops.java get generated in a bit different order for me, but it does that with 2.3.0 as well.

diff --git a/tensorflow-core/tensorflow-core-api/src/gen/annotations/org/tensorflow/op/Ops.java b/tensorflow-core/tensorflow-core-api/src/gen/annotations/org/tensorflow/op/Ops.java
index 8c7aa1c0..bad63582 100644
--- a/tensorflow-core/tensorflow-core-api/src/gen/annotations/org/tensorflow/op/Ops.java
+++ b/tensorflow-core/tensorflow-core-api/src/gen/annotations/org/tensorflow/op/Ops.java
@@ -347,10 +347,10 @@ public final class Ops {
 
   public final SignalOps signal;
 
-  public final QuantizationOps quantization;
-
   public final TrainOps train;
 
+  public final QuantizationOps quantization;
+
   private final Scope scope;
 
   private Ops(Scope scope) {
@@ -372,8 +372,8 @@ public final class Ops {
     math = new MathOps(scope);
     audio = new AudioOps(scope);
     signal = new SignalOps(scope);
-    quantization = new QuantizationOps(scope);
     train = new TrainOps(scope);
+    quantization = new QuantizationOps(scope);
   }
 
   /**

@Craigacp
Copy link
Collaborator

Craigacp commented Oct 1, 2020

Weird. I thought I'd fixed the generation order. Must be another place that isn't using a LinkedHashMap.

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.

Upgrade to TF 2.3.1
3 participants