search

StanJulia / CmdStan.jl

CmdStan.jl v6 provides an alternative, older Julia wrapper to Stan's `cmdstan` executable. CmdStan will be deprecated in 2022.
MIT License
30 stars 12 forks source link

Method definition overwritten in module StanMCMCChains #37

Closed jeremiahpslewis closed 5 years ago

jeremiahpslewis commented 5 years ago

Hi! When I install CmdStan, I get the following error:

[ Info: Generating builtins for this julia version...
WARNING: Method definition convert_a3d(Any, Any, Base.Val{:mcmcchains}) in module CmdStan at /opt/julia/packages/CmdStan/9uCTO/src/utilities/convert_a3d.jl:54 overwritten in module StanMCMCChains at /opt/julia/packages/StanMCMCChains/kuO0o/src/utilities/convert_a3d.jl:6.
┌ Warning: `getindex(o::PyObject, s::AbstractString)` is deprecated in favor of dot overloading (`getproperty`) so elements should now be accessed as e.g. `o."s"` instead of `o["s"]`.
│   caller = top-level scope at none:0
└ @ Core none:0
WARNING: Method definition convert_a3d(Any, Any, Base.Val{:mcmcchains}) in module CmdStan at /opt/julia/packages/CmdStan/9uCTO/src/utilities/convert_a3d.jl:54 overwritten in module StanMCMCChains at /opt/julia/packages/StanMCMCChains/kuO0o/src/utilities/convert_a3d.jl:6.

I'd love to create a pull request to fix this issue, but I'm new to Julia and don't know where to start on figuring it out. Cheers, /j

Version Info:

Julia Version 1.1.0
Commit 80516ca202 (2019-01-21 21:24 UTC)
Platform Info:
  OS: Linux (x86_64-pc-linux-gnu)
  CPU: Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-6.0.1 (ORCJIT, broadwell)
Environment:
  JULIA_DEPOT_PATH = /opt/julia
  JULIA_PKGDIR = /opt/julia
  JULIA_VERSION = 1.1.0
  JULIA_CMDSTAN_HOME = /home/jovyan/cmdstan
goedman commented 5 years ago

Hi Jeremiah,

That problem should be fixed in the latest master but I’ve held back releasing that fix to test it with some updates to MCMCChains. Hopefully I can merge those today.

Do the examples run or ‘Pkg.test(“CmdStan”)’?

Best, Rob

jeremiahpslewis commented 5 years ago

Hi Rob,

Great to hear; it's not causing any actual issues, so happy to wait for the next release.

goedman commented 5 years ago

@jlewis91 Just filed the request for the update to v5.0.1 as MCMCChains v0.3.5 is now available. Should be accepted in the next few hours. Thanks for raising the issue.