timholy / SnoopCompile.jl

Provide insights about latency (TTFX) for Julia packages
https://timholy.github.io/SnoopCompile.jl/dev/
Other
309 stars 48 forks source link

ERROR: LoadError: ArgumentError: Package SnoopCompileBot not found in current path #111

Closed daschw closed 4 years ago

daschw commented 4 years ago

I am trying to use SnoopCompileBot via Github Actions for Plots.jl and get the following error:


Run julia --project -e 'include("deps/SnoopCompile/snoop_bot.jl")'
[ Info: /home/runner/work/Plots.jl/Plots.jl/src/precompile_includer.jl file will be created/overwritten
[ Info: SnoopCompile will try to write  "include("precompile_includer.jl")" before end of the module in /home/runner/work/Plots.jl/Plots.jl/src/Plots.jl. Assume that the last `end` is the end of a module.
ERROR: LoadError: ArgumentError: Package SnoopCompileBot not found in current path:
- Run `import Pkg; Pkg.add("SnoopCompileBot")` to install the SnoopCompileBot package.

Stacktrace:
 [1] require(::Module, ::Symbol) at ./loading.jl:892
 [2] top-level scope at /home/runner/.julia/packages/SnoopCompileBot/K9naN/src/snoop_bot.jl:112
 [3] eval at ./boot.jl:331 [inlined]
 [4] snoop_bot(::BotConfig, ::String, ::Module; snoop_mode::Symbol) at /home/runner/.julia/packages/SnoopCompileBot/K9naN/src/snoop_bot.jl:182
 [5] snoop_bot at /home/runner/.julia/packages/SnoopCompileBot/K9naN/src/snoop_bot.jl:177 [inlined] (repeats 2 times)
 [6] top-level scope at /home/runner/work/Plots.jl/Plots.jl/deps/SnoopCompile/snoop_bot.jl:3
 [7] include(::String) at ./client.jl:439
 [8] top-level scope at none:1
in expression starting at /home/runner/work/Plots.jl/Plots.jl/deps/SnoopCompile/snoop_bot.jl:3
##[error]Process completed with exit code 1.

I think I followed the instructions in the docs but I am probably doing something stupid. Here's the test PR on my fork: https://github.com/daschw/Plots.jl/pull/2. I would really appreciate some hints to what I am doing wrong. Thanks!

aminya commented 4 years ago

Sorry for this issue. Recently, we switched to a cutting edge way of package management, and we still have some problems here and there. We are working on it. You can see the discussions here #105.

timholy commented 4 years ago

@daschw, SnoopCompile 1.6.2 should fix this, I'd be curious to hear your experience.

daschw commented 4 years ago

Thanks, I'm already working on it. One thing I just ran into:

BotConfig(..., version = [v"1.3", v"1", "nightly"])

gives

ERROR: LoadError: TypeError: in keyword argument version, expected Union{Nothing, Array{#s18,1} where #s18<:Union{VersionNumber, String}}, got Array{Any,1}

which I think could be supported. version = ["1.3", "1", "nightly"] seems to work though.