Skip to content
This repository has been archived by the owner on Oct 18, 2023. It is now read-only.

java.lang.UnsatisfiedLinkError in 0.5.5 with TF 1.14.0 - Reason: image not found #193

Closed
nevillelyh opened this issue Oct 1, 2019 · 9 comments
Closed

java.lang.UnsatisfiedLinkError in 0.5.5 with TF 1.14.0 - Reason: image not found #193

nevillelyh opened this issue Oct 1, 2019 · 9 comments
Labels
bug Something isn't working

Comments

@nevillelyh
Copy link
Contributor

Details see: spotify/scio#2254 (comment)
@nevillelyh nevillelyh added the bug Something isn't working label Oct 1, 2019
@mattnworb
Copy link
Member

mattnworb commented Oct 18, 2019 •
edited
Loading

I’d like to tackle this next week now that TF 0.15 is out

@mattnworb
Copy link
Member

tensorflow/tensorflow#30635 (comment)

@regadas
Copy link
Contributor

regadas commented Oct 18, 2019 •
edited
Loading

zoltar works around the issue with loading native lib but if there's another tensorflow in the classpath the above error can happen;

With TF 1.15.0 that's supposedly fixed so no need for the custom stuff;

(sorry for the late reply ... on parental leave 😄 )

@regadas
Copy link
Contributor

regadas commented Oct 18, 2019

if it's really fixed in TF 1.15.0 we should revert this ... #190

@mattnworb
Copy link
Member

not sure if anything has actually changed, https://github.com/tensorflow/tensorflow/blob/v1.15.0/tensorflow/java/src/main/java/org/tensorflow/NativeLibrary.java looks unchanged since v1.14

@mattnworb
Copy link
Member

I misspoke above, tensorflow/tensorflow@a4a4057 is tagged with v1.15.0 and v2.0.0

@mattnworb
Copy link
Member

and org.tensorflow:tensorflow version 1.15.0 is not in Maven Central yet (nor is 2.0.0): tensorflow/tensorflow#32943

@mattnworb mattnworb mentioned this issue Oct 21, 2019
Merged
@mattnworb
Copy link
Member

I am going to try to test 0.5.6-SNAPSHOT with #196 included against spotify/scio#2254 (comment) so we can be sure the root issue is fixed by upgrading to tensorflow 1.15 before cutting a new release of zoltar and closing this

@mattnworb
Copy link
Member

tested a locally built snapshot of zoltar 0.5.6-SNAPSHOT against scio master (e82c4329) as in spotify/scio#2254 (comment), tests are green ✅

mattnworb added a commit to mattnworb/scio that referenced this issue Oct 23, 2019
@mattnworb
upgrade zoltar to 0.5.6
99d3dc1 
This re-applies spotify#2254 now that
zoltar is using Tensorflow 1.15
(spotify/zoltar#193)
@mattnworb mattnworb mentioned this issue Oct 23, 2019
Merged
regadas pushed a commit to spotify/scio that referenced this issue Oct 24, 2019
@mattnworb @regadas
upgrade zoltar to 0.5.6 (#2318)
a258543 
This re-applies #2254 now that
zoltar is using Tensorflow 1.15
(spotify/zoltar#193)
regadas pushed a commit to spotify/scio that referenced this issue Oct 28, 2019
@mattnworb @regadas
upgrade zoltar to 0.5.6 (#2318)
9fa7682 
This re-applies #2254 now that
zoltar is using Tensorflow 1.15
(spotify/zoltar#193)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mattnworb @regadas @nevillelyh