search

KnetML / MAC-Network

Memory, Attention and Composition (MAC) Network for CLEVR implemented via KnetLayers
MIT License
6 stars 4 forks source link

Package versions in Manifest may not be compatible with Julia 1.1 #3

Closed denizyuret closed 5 years ago

denizyuret commented 5 years ago

I get the following error in cell 5:

Error encountered while loading "/mnt/efs/comp541/MAC-Network/data/demo/CLEVR_v1.0/images/val/CLEVR_val_013382.png".
Fatal error:
rethrow(exc) not allowed outside a catch block

Stacktrace:
 [1] rethrow(::InitError) at ./error.jl:52
 [2] handle_error(::InitError, ::FileIO.File{FileIO.DataFormat{:PNG}}) at /home/ec2-user/.julia/packages/FileIO/lt9I0/src/error_handling.jl:80
 [3] handle_exceptions(::Array{Any,1}, ::String) at /home/ec2-user/.julia/packages/FileIO/lt9I0/src/error_handling.jl:75
 [4] #load#27(::Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}}, ::Function, ::FileIO.File{FileIO.DataFormat{:PNG}}) at /home/ec2-user/.julia/packages/FileIO/lt9I0/src/loadsave.jl:193
 [5] load(::FileIO.File{FileIO.DataFormat{:PNG}}) at /home/ec2-user/.julia/packages/FileIO/lt9I0/src/loadsave.jl:172
 [6] #load#13(::Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}}, ::Function, ::String) at /home/ec2-user/.julia/packages/FileIO/lt9I0/src/loadsave.jl:118
 [7] load(::String) at /home/ec2-user/.julia/packages/FileIO/lt9I0/src/loadsave.jl:118
 [8] top-level scope at In[5]:1
denizyuret commented 5 years ago

Apparently ImageMagick needs to be loaded first before anything else. PR #4 fixes this.

ekinakyurek commented 5 years ago

Thank you. I think you also have different updates for visualization in this : https://github.com/KnetML/NLPdemos/commit/6b8d8a416ca18c1e1642d6b91efef4a8c5874782

Could you add them too?

On Sun, Feb 3, 2019 at 10:18 AM denizyuret notifications@github.com wrote:

Apparently ImageMagick needs to be loaded first before anything else. PR

4 https://github.com/KnetML/MAC-Network/pull/4 fixes this.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/KnetML/MAC-Network/issues/3#issuecomment-460060491, or mute the thread https://github.com/notifications/unsubscribe-auth/AOpr8eYdt0w4jImbVEHMy7pYpI2XXY1Lks5vJv3NgaJpZM4agH9R .

ekinakyurek commented 5 years ago

Oh, I think you’ve done already?

On Sun, Feb 3, 2019 at 10:55 AM Ekin Akyürek akyurekekin@gmail.com wrote:

Thank you. I think you also have different updates for visualization in this :

https://github.com/KnetML/NLPdemos/commit/6b8d8a416ca18c1e1642d6b91efef4a8c5874782

Could you add them too?

On Sun, Feb 3, 2019 at 10:18 AM denizyuret notifications@github.com wrote:

Apparently ImageMagick needs to be loaded first before anything else. PR

4 https://github.com/KnetML/MAC-Network/pull/4 fixes this.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/KnetML/MAC-Network/issues/3#issuecomment-460060491, or mute the thread https://github.com/notifications/unsubscribe-auth/AOpr8eYdt0w4jImbVEHMy7pYpI2XXY1Lks5vJv3NgaJpZM4agH9R .

denizyuret commented 5 years ago

Yes, I changed the visualize() to match NLPdemos.

On Sun, Feb 3, 2019 at 6:57 PM Ekin Akyürek notifications@github.com wrote:

Oh, I think you’ve done already?

On Sun, Feb 3, 2019 at 10:55 AM Ekin Akyürek akyurekekin@gmail.com wrote:

Thank you. I think you also have different updates for visualization in this :

https://github.com/KnetML/NLPdemos/commit/6b8d8a416ca18c1e1642d6b91efef4a8c5874782

Could you add them too?

On Sun, Feb 3, 2019 at 10:18 AM denizyuret notifications@github.com wrote:

Apparently ImageMagick needs to be loaded first before anything else. PR

4 https://github.com/KnetML/MAC-Network/pull/4 fixes this.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <https://github.com/KnetML/MAC-Network/issues/3#issuecomment-460060491 , or mute the thread < https://github.com/notifications/unsubscribe-auth/AOpr8eYdt0w4jImbVEHMy7pYpI2XXY1Lks5vJv3NgaJpZM4agH9R

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/KnetML/MAC-Network/issues/3#issuecomment-460063588, or mute the thread https://github.com/notifications/unsubscribe-auth/ABvNprLiWrDGfMSUn3bL04itjppLiiLfks5vJwbLgaJpZM4agH9R .

ekinakyurek commented 5 years ago

solved with PR#4