Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plots/HDF5: Add dependency #101

Closed
miili opened this issue Nov 9, 2022 · 3 comments
Closed

Plots/HDF5: Add dependency #101

miili opened this issue Nov 9, 2022 · 3 comments

Comments

@miili
Copy link

miili commented Nov 9, 2022

On Julia 1.8.2 I run into dependency issues wrangling around hdf5. Plots is not tied in Project.toml, I am not used to Julia and this is a wild guess.

WARNING: could not import HDF5.Group into _hdf5_implementation
WARNING: could not import HDF5.Dataset into _hdf5_implementation
┌ Warning: Error requiring `HDF5` from `Plots`
│   exception =
│    LoadError: UndefVarError: Group not defined
│    Stacktrace:
│      [1] top-level scope
│        @ ~/.julia/packages/Plots/wutJB/src/backends/hdf5.jl:138
│      [2] include(mod::Module, _path::RelocatableFolders.Path)
│        @ Base ./Base.jl:419
│      [3] include(x::RelocatableFolders.Path)
│        @ Plots ~/.julia/packages/Plots/wutJB/src/Plots.jl:1
│      [4] top-level scope
│        @ ~/.julia/packages/Plots/wutJB/src/init.jl:68
│      [5] eval
│        @ ./boot.jl:368 [inlined]
│      [6] eval
│        @ ~/.julia/packages/Plots/wutJB/src/Plots.jl:1 [inlined]
│      [7] (::Plots.var"#360#405")()
│        @ Plots ~/.julia/packages/Requires/Z8rfN/src/require.jl:101
│      [8] macro expansion
│        @ ./timing.jl:382 [inlined]
│      [9] err(f::Any, listener::Module, modname::String, file::String, line::Any)
│        @ Requires ~/.julia/packages/Requires/Z8rfN/src/require.jl:47
│     [10] (::Plots.var"#359#404")()
│        @ Plots ~/.julia/packages/Requires/Z8rfN/src/require.jl:100
│     [11] withpath(f::Any, path::String)
│        @ Requires ~/.julia/packages/Requires/Z8rfN/src/require.jl:37
│     [12] (::Plots.var"#358#403")()
│        @ Plots ~/.julia/packages/Requires/Z8rfN/src/require.jl:99
│     [13] listenpkg(f::Any, pkg::Base.PkgId)
│        @ Requires ~/.julia/packages/Requires/Z8rfN/src/require.jl:20
│     [14] macro expansion
│        @ ~/.julia/packages/Requires/Z8rfN/src/require.jl:98 [inlined]
│     [15] __init__()
│        @ Plots ~/.julia/packages/Plots/wutJB/src/init.jl:67
│     [16] _include_from_serialized(pkg::Base.PkgId, path::String, depmods::Vector{Any})
│        @ Base ./loading.jl:831
│     [17] _require_search_from_serialized(pkg::Base.PkgId, sourcepath::String, build_id::UInt64)
│        @ Base ./loading.jl:1039
│     [18] _require(pkg::Base.PkgId)
│        @ Base ./loading.jl:1315
│     [19] _require_prelocked(uuidkey::Base.PkgId)
│        @ Base ./loading.jl:1200
│     [20] macro expansion
│        @ ./loading.jl:1180 [inlined]
│     [21] macro expansion
│        @ ./lock.jl:223 [inlined]
│     [22] require(into::Module, mod::Symbol)
│        @ Base ./loading.jl:1144
│     [23] include(mod::Module, _path::String)
│        @ Base ./Base.jl:419
│     [24] exec_options(opts::Base.JLOptions)
│        @ Base ./client.jl:303
│     [25] _start()
│        @ Base ./client.jl:522in expression starting at /home/marius/.julia/packages/Plots/wutJB/src/backends/hdf5.jl:35
└ @ Requires ~/.julia/packages/Requires/Z8rfN/src/require.jl:51

HDF5 update is held back by SeisNoise, thus causing incomp w newer versions of Plots

@tclements
Copy link
Collaborator

Thanks for bringing this to my attention! I believe this is coming from HDF5.jl not being up to date in SeisIO.jl. I can push a fix to my fork of SeisIO.jl and start using that fork of SeisIO.jl that in SeisNoise.jl.

@tclements
Copy link
Collaborator

A quick fix for this would be to add my fork of SeisIO.jl, then install SeisNoise.jl. We are currently working on migrating from SeisIO.jl to https://github.com/SeismoJulia/SeisBase.jl.

pkg> rm SeisIO.jl
pkg> rm SeisNoise.jl
pkg> add https://github.com/tclements/SeisIO.jl.git
pkg> add SeisNoise.jl

Let me know if that works, thanks!

@miili
Copy link
Author

miili commented Nov 16, 2022

Hi Tim, I resolved the dependency conflict by enforcing a downgrade. I am no expert in Julia and had it working quickly. Maybe this is not a critical issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants