eaplatanios / tensorflow_scala

TensorFlow API for the Scala Programming Language
http://platanios.org/tensorflow_scala/
Apache License 2.0
936 stars 96 forks source link

Native jni libs contain only darwin artifacts #188

Closed mandar2812 closed 3 years ago

mandar2812 commented 3 years ago

When I import tf scala 0.5.4 in my DynaML build, it fails to load the JNI library. After inspecting the jar file I found that it contains bindings only for Mac, but not for Linux or windows. I have built my code on an Ubuntu PC

❯ jar -tf ./target/universal/stage/lib/org.platanios.tensorflow-jni_2.12-0.5.4.jar | grep "lib"
native/darwin-x86_64/libtensorflow_jni.so
native/darwin-x86_64/libtensorflow_ops.so
eaplatanios commented 3 years ago

Thanks for letting me know @mandar2812 ! I've messed up this release because of how I tried adding support for windows. Could you please try again using 0.5.5?

steve-gailey-exa commented 3 years ago

Sure will do and thanks for your work on this project - I for one really appreciate it. Do you have a funding page?

S.

[X] Steve Gailey | Solutions Architecture steve.gailey@exabeam.commailto:steve.gailey@exabeam.com | +44 7795 220392 exabeam.comhttps://www.exabeam.com/ | facebookhttps://www.facebook.com/exabeam/ | linkedinhttps://www.linkedin.com/company/exabeam | twitterhttps://twitter.com/exabeam

[X]https://www.gartner.com/reviews/market/security-information-event-management/vendor/Exabeam/?pid=27389%20target=_blank%20rel=noopener

On 24 Sep 2020, at 19:09, Anthony Platanios notifications@github.com<mailto:notifications@github.com> wrote:

Thanks for letting me know @mandar2812https://github.com/mandar2812 ! I've messed up this release because of how I tried adding support for windows. Could you please try again using 0.5.5?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/eaplatanios/tensorflow_scala/issues/188#issuecomment-698502639, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOVZTRGS25L2V2UCJN2JPCLSHODMHANCNFSM4RYRLXDQ.

mandar2812 commented 3 years ago

@eaplatanios It works now! Thanks for looking into this.