Skip to content
This repository was archived by the owner on Apr 19, 2026. It is now read-only.

Updated for 0.2 version - #18

Merged
karllessard merged 4 commits into
tensorflow:masterfrom
zaleslaw:issue-17
Oct 10, 2020
Merged

Updated for 0.2 version#18
karllessard merged 4 commits into
tensorflow:masterfrom
zaleslaw:issue-17

Conversation

@zaleslaw

@zaleslaw zaleslaw commented Oct 9, 2020

Copy link
Copy Markdown
Contributor

It's a draft PR, a few examples works incorrectly.

Operand batchLoss = tf.nn
.softmaxCrossEntropyWithLogits(logits, oneHot, -1); - behaviour was changed.

Now it fails with exception

Exception in thread "main" org.tensorflow.exceptions.TFInvalidArgumentException: Dimension must be 2 but is 1 for '{{node SoftmaxCrossEntropyWithLogits/Transpose_1}} = Transpose[T=DT_FLOAT, Tperm=DT_INT64](Add_1, SoftmaxCrossEntropyWithLogits/Concat_1)' with input shapes: [?,10], [1].
	at org.tensorflow.internal.c_api.AbstractTF_Status.throwExceptionIfNotOK(AbstractTF_Status.java:87)
	at org.tensorflow.GraphOperationBuilder.finish(GraphOperationBuilder.java:370)
	at org.tensorflow.GraphOperationBuilder.build(GraphOperationBuilder.java:78)
	at org.tensorflow.GraphOperationBuilder.build(GraphOperationBuilder.java:57)
	at org.tensorflow.op.linalg.Transpose.create(Transpose.java:56)
	at org.tensorflow.op.nn.SoftmaxCrossEntropyWithLogits.moveDimToEnd(SoftmaxCrossEntropyWithLogits.java:211)
	at org.tensorflow.op.nn.SoftmaxCrossEntropyWithLogits.softmaxCrossEntropyWithLogits(SoftmaxCrossEntropyWithLogits.java:104)
	at org.tensorflow.op.NnOps.softmaxCrossEntropyWithLogits(NnOps.java:1873)
	at org.tensorflow.model.examples.cnn.vgg.VGGModel.buildFCLayersAndRegularization(VGGModel.java:163)
	at org.tensorflow.model.examples.cnn.vgg.VGGModel.compile(VGGModel.java:124)
	at org.tensorflow.model.examples.cnn.vgg.VGGModel.<init>(VGGModel.java:76)
	at org.tensorflow.model.examples.cnn.vgg.VGG11OnFashionMNIST.main(VGG11OnFashionMNIST.java:43)

@google-cla google-cla Bot added the cla: yes label Oct 9, 2020
@zaleslaw

Copy link
Copy Markdown
Contributor Author

@karllessard Please, review and merge (no radical updates here) (related to #18 issue)

@karllessard
karllessard merged commit c3f6dbf into tensorflow:master Oct 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants