-
Notifications
You must be signed in to change notification settings - Fork 74k
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
Java Module support #39378
Comments
@cowwoc Could you please let us know if this issue still persists ? If it it resolved then please feel free to move this issue to close status ?Thanks! |
@kumariko Yes, the issue still persists. |
@cowwoc, |
@gadagashwini What are you asking? Did you add module-info.java to Tensorflow? |
@cowwoc, As you can see https://www.tensorflow.org/install/lang_java_legacy#tensorflow_with_the_jdk Tensorflow has Java compiler. Thanks! |
@gadagashwini Sure, but this issue is about "Java Module" support which is about a lot more than just providing Java support. |
Hi @cowwoc ! Could you check the run time from Tensorflow/Java for your usecase. Thank you! |
Sorry @mohantym I'm not working on any AI projects at the moment. Can anyone else test? |
@cowwoc ! Other language binding and signature builds are being supported through Tensorflow/Build and Tensorflow/Community. Thank you! |
I don't think you understand what Java Module (JPMS) is. It's not enough to say that Tensorflow/Java can run under a JVM. You'll need to create a new application that uses JPMS and import TensorFlow in your |
@cowwoc ! Thanks for the update. will let you know once i have some update on above pointers. Thank you! |
Describe the problem
Applications using Java9+ benefit from libraries have Java Module (JPMS) support. This comes in two stages:
module-info.java
to the library. Either publish a separate artifact for Java9+ or simply addMETA-INF/versions/9/module-info.class
to support both versions simultaneously (multi-release jar)module-info.java
yet in which case you can simply addAutomatic-Module-Name
to the manifest file and begin resolving the split package issue.The text was updated successfully, but these errors were encountered: