Skip to content
This repository has been archived by the owner. It is now read-only.

Fix mvn build compile protobuf error #3

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
1 participant
@lgrcyanny
Copy link

commented Dec 19, 2017

What changes were proposed in this pull request?

1. Fix mvn build error
build the master branch, I got many errors as follows:
jpmml-tensorflow/target/generated-sources/protobuf/java/org/tensorflow/framework/TensorProto.java:[63,18] Can't Find Symbol

2. Fix a compile error in jpmml-tensorflow/src/main/java/org/jpmml/tensorflow/SavedModel.java
The error is
/Users/lgrcyanny/Codecookies/machine-learning-workspace/pmml-wp/jpmml-tensorflow/src/main/java/org/jpmml/tensorflow/SavedModel.java:[136,50] incompatiable with: java.util.List<org.tensorflow.Tensor<?>> can't transformed to java.util.List<org.tensorflow.Tensor>

The reason is tensorflow latest API is 1.4

How was this patch tested?

build on my local MAC machine and success

  • protoc version is 3.5.0
  • jdk 1.8
  • tensorlfow 1.4
@lgrcyanny

This comment has been minimized.

Copy link
Author

commented Dec 19, 2017

@vruusmann please review the patch, thank you.

And I found that tensorflow support is not complete yet, such as it doesn't support KMeans, RNN, SVM etc. are there any plans to support more tensorflow estimators?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.