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

Problems installing the package #311

Closed
plcrodrigues opened this issue Jul 31, 2024 · 6 comments · Fixed by #312
Closed

Problems installing the package #311

plcrodrigues opened this issue Jul 31, 2024 · 6 comments · Fixed by #312

Comments

@plcrodrigues
Copy link

Hi, everyone. After reading the "Calibrate, Emulate, Sample" paper by Cleary et al. I wanted to play a bit with the algorithms and get a better grasp of the full methodology. I was, therefore, vey happy to see that the paper came with a well structured documentation. Kudos for that! Unfortunately, I was unable to install the package on a newly created environment on my MacBook Air with ARM architecture.

The error that I get after doing

julia --project
> ]
> add Conda
> add CalibrateEmulateSample

is the following

PkgPrecompileError: The following 1 direct dependency failed to precompile:

CalibrateEmulateSample [95e48a1f-0bec-4818-9538-3db4340308e3]

Failed to precompile CalibrateEmulateSample [95e48a1f-0bec-4818-9538-3db4340308e3] to "/Users/plcrodrigues/.julia/compiled/v1.10/CalibrateEmulateSample/jl_Hmh8kk".
WARNING: could not import EnsembleKalmanProcesses.Observations into CalibrateEmulateSample
ERROR: LoadError: UndefVarError: `Observations` not defined
Stacktrace:
 [1] include(mod::Module, _path::String)
   @ Base ./Base.jl:495
 [2] include(x::String)
   @ CalibrateEmulateSample ~/.julia/packages/CalibrateEmulateSample/mK8b2/src/CalibrateEmulateSample.jl:8
 [3] top-level scope
   @ ~/.julia/packages/CalibrateEmulateSample/mK8b2/src/CalibrateEmulateSample.jl:19
 [4] include
   @ ./Base.jl:495 [inlined]
 [5] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String}, load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt128}}, source::Nothing)
   @ Base ./loading.jl:2222
 [6] top-level scope
   @ stdin:3
in expression starting at /Users/plcrodrigues/.julia/packages/CalibrateEmulateSample/mK8b2/src/Utilities.jl:1
in expression starting at /Users/plcrodrigues/.julia/packages/CalibrateEmulateSample/mK8b2/src/CalibrateEmulateSample.jl:1
in expression starting at stdin:

Am I missing something? (disclaimer: I have near to zero experience with Julia language...)

Thanks for your attention!

@plcrodrigues
Copy link
Author

Ah, I guess it must be connected to this : CliMA/EnsembleKalmanProcesses.jl@dab5aff

I installed version 1.1.5 of EnsembleKalmanProcesses and it worked fine.

@odunbar
Copy link
Collaborator

odunbar commented Jul 31, 2024

Apologies! Just working on this one - Yeah we overhauled the Observations. I messed up doing an EKP release of v1.1.X where I should have done v1.2.1. This would prevent CES automatically updating to a breaking change!

@odunbar
Copy link
Collaborator

odunbar commented Jul 31, 2024

If you could please fix your version to v1.1.5 for now and this should be resolved ASAP! Thanks for reporting this. As the EKP change was last night, and I am working on the CES compat today - I am afraid you hit the perfect storm.

@plcrodrigues
Copy link
Author

No problem, it forced me to try to understand a bit more of Julia and its inner workings :)

@odunbar
Copy link
Collaborator

odunbar commented Aug 1, 2024

OK the fix is in!
Currently this is just on main, so

  • if you are installing CES locally you can use git pull on main to get the changes.
  • if you are using the package manager to add CES, add CalibrateEmulateSample#main instead of just add CalibrateEmulateSample or just dev your local version
  • ensure in either case that the EnsembleKalmanProcesses version is updated to 1.1.7 else call update in Pkg

Hopefully in the next few days I'll release a new version so just the add works. (I am still wanting to merge one other PR (#292) before releasing a new version)

@odunbar
Copy link
Collaborator

odunbar commented Aug 1, 2024

Please re-open the issue or create more issues if you have other problems! Thanks for helping improve the code :)

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

Successfully merging a pull request may close this issue.

2 participants