JagsJulia / Jags.jl

Julia package for using Jags as an external program
Other
17 stars 5 forks source link

Fix crash when updating bugs file #26

Closed hellemo closed 4 years ago

hellemo commented 4 years ago

I ran into this while updating my old project using Jags: while updating the bugs file, Jags.jl is trying to rm the just opened file, which makes the code crash.

I also added test dependencies to be able to run the tests under julia v1.4 (they all seem to run without problems locally).

goedman commented 4 years ago

Thanks Lars,

Added a few more updates to Project.toml and pushed a new version.

Rob J Goedman goedman@icloud.com

On Apr 15, 2020, at 16:41, hellemo notifications@github.com wrote:

I ran into this while updating my old project using Jags: while updating the bugs file, Jags.jl is trying to rm the just opened file, which makes the code crash.

I also added test dependencies to be able to run the tests under julia v1.4 (they all seem to run without problems locally).

You can view, comment on, or merge this pull request online at:

https://github.com/JagsJulia/Jags.jl/pull/26 https://github.com/JagsJulia/Jags.jl/pull/26 Commit Summary

Fix crash when trying to rm open file. Add test dependencies to run in julia v1.4. File Changes

M src/jagsmodel.jl https://github.com/JagsJulia/Jags.jl/pull/26/files#diff-3ed08fad31521f224ff55f9f857381bd (4) A test/Project.toml https://github.com/JagsJulia/Jags.jl/pull/26/files#diff-0e4d6ad45c2cb92450b5a25586d76438 (6) Patch Links:

https://github.com/JagsJulia/Jags.jl/pull/26.patch https://github.com/JagsJulia/Jags.jl/pull/26.patch https://github.com/JagsJulia/Jags.jl/pull/26.diff https://github.com/JagsJulia/Jags.jl/pull/26.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/JagsJulia/Jags.jl/pull/26, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAMX2B4WFDJCGSKOZ4BVGDRMYLXJANCNFSM4MI4KSEQ.