JuliaOcean / Climatology.jl

Climatology and the Ocean state
https://JuliaOcean.github.io/Climatology.jl/dev
MIT License
12 stars 2 forks source link

IOError in get_from_dataverse #10

Closed visr closed 3 years ago

visr commented 3 years ago

Running the tests of IndividualDisplacements master and latest release of OceanStateEstimation on Julia 1.5 on Windows is get this error:

ERROR: LoadError: IOError: could not spawn `mv 'C:\Users\VISSER~1\AppData\Local\Temp\jl_nbVsVs0QNz' 'C:\Users\visser_mn\.julia\artifacts\ec3707d05425e2b371b5cb619383db1a3a9ab3de/VV
ELMASS/VVELMASS.0001.nc'`: no such file or directory (ENOENT)
Stacktrace:
 [1] _spawn_primitive(::String, ::Cmd, ::Array{Any,1}) at .\process.jl:99
 [2] #585 at .\process.jl:112 [inlined]
 [3] setup_stdios(::Base.var"#585#586"{Cmd}, ::Array{Any,1}) at .\process.jl:196
 [4] _spawn at .\process.jl:111 [inlined]
 [5] run(::Cmd; wait::Bool) at .\process.jl:439
 [6] run at .\process.jl:438 [inlined]
 [7] get_from_dataverse(::String, ::String, ::String) at C:\Users\visser_mn\.julia\packages\OceanStateEstimation\jbcDP\src\OceanStateEstimation.jl:37
 [8] get_ecco_velocity_if_needed() at C:\Users\visser_mn\.julia\dev\IndividualDisplacements\src\read.jl:166
 [9] top-level scope at C:\Users\visser_mn\.julia\dev\IndividualDisplacements\test\runtests.jl:5
 [10] include(::String) at .\client.jl:457
 [11] top-level scope at none:6
in expression starting at C:\Users\visser_mn\.julia\dev\IndividualDisplacements\test\runtests.jl:5
ERROR: Package IndividualDisplacements errored during testing

Which is triggered here: https://github.com/gaelforget/OceanStateEstimation.jl/blob/cf1fd8b78855c9bad5ee274fc65e1cc520d9b258/src/OceanStateEstimation.jl#L35-L37

Not sure what happened, the Temp source directory was not there or already removed. Maybe it is better to replace the run function by the built in mv function as well?

Part of https://github.com/openjournals/joss-reviews/issues/2813.

gaelforget commented 3 years ago

Thanks for pointing this out.

Should be fixed now with new https://github.com/gaelforget/OceanStateEstimation.jl/releases/tag/v0.1.7 where I did the mv change and added windows CI that was missing (see https://travis-ci.org/github/gaelforget/OceanStateEstimation.jl/builds/743850393)

Unfortunately I do not personally have access to a windows machine and dont seem able to add it to IndividualDisplacements.jl 's Travis CI due to failure building Plots.jl (see https://travis-ci.org/github/JuliaClimate/IndividualDisplacements.jl/builds/743858759).

Could you please give this another try with v0.1.7 of OceanStateEstimation.jl and let me know if you still run into an issue like this?

ps. I did a couple updates / fixes in IndividualDisplacements.jl , am waiting for CI to complete before registering new version later today

visr commented 3 years ago

Ok thanks, will try it later!

The Travis CI issue seems to be this: https://github.com/JuliaIO/FFMPEG.jl/issues/30#issuecomment-706531526

visr commented 3 years ago

This is working now for me, thanks!