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

Require DataFrames failed #169

Closed
zhouyan opened this issue Jan 29, 2018 · 6 comments
Closed

Require DataFrames failed #169

zhouyan opened this issue Jan 29, 2018 · 6 comments

Comments

@zhouyan
Copy link

zhouyan commented Jan 29, 2018

I am getting the following warning when loading PlotlyJS after recent update. I do have DataFrames installed and it works fine. I can't make sense of the error message though. All appearance of df in dataframe_api.jl looks fine to me and the line in question according to the error message 522 does not exists

WARNING: Error requiring DataFrames from PlotlyJS:
LoadError: UndefVarError: df not defined
Stacktrace:
 [1] include_from_node1(::String) at ./loading.jl:576
 [2] include(::String) at ./sysimg.jl:14
 [3] err(::PlotlyJS.##264#271, ::Module, ::String) at /opt/fairtide/tools/pkgs/julia/0.6.2/share/julia/site/v0.6/Requires/src/require.jl:42
 [4] withpath(::PlotlyJS.##263#270, ::String) at /opt/fairtide/tools/pkgs/julia/0.6.2/share/julia/site/v0.6/Requires/src/require.jl:32
 [5] listenmod(::PlotlyJS.##262#269, ::Symbol) at /opt/fairtide/tools/pkgs/julia/0.6.2/share/julia/site/v0.6/Requires/src/require.jl:13
 [6] macro expansion at /opt/fairtide/tools/pkgs/julia/0.6.2/share/julia/site/v0.6/Requires/src/require.jl:52 [inlined]
 [7] (::PlotlyJS.##261#268)() at /opt/fairtide/tools/pkgs/julia/0.6.2/share/julia/site/v0.6/Requires/src/init.jl:17
 [8] __init__() at /opt/fairtide/tools/pkgs/julia/0.6.2/share/julia/site/v0.6/Requires/src/init.jl:24
 [9] _include_from_serialized(::String) at ./loading.jl:157
 [10] _require_from_serialized(::Int64, ::Symbol, ::String, ::Bool) at ./loading.jl:200
 [11] _require_search_from_serialized(::Int64, ::Symbol, ::String, ::Bool) at ./loading.jl:236
 [12] _require(::Symbol) at ./loading.jl:441
 [13] require(::Symbol) at ./loading.jl:405
 [14] include_from_node1(::String) at ./loading.jl:576
 [15] include(::String) at ./sysimg.jl:14
 [16] eval(::Module, ::Any) at ./boot.jl:235
 [17] eval_user_input(::Any, ::Base.REPL.REPLBackend) at ./REPL.jl:66
 [18] macro expansion at ./REPL.jl:97 [inlined]
 [19] (::Base.REPL.##1#2{Base.REPL.REPLBackend})() at ./event.jl:73
while loading /opt/fairtide/tools/pkgs/julia/0.6.2/share/julia/site/v0.6/PlotlyJS/src/dataframes_api.jl, in expression starting on line 522
@sglyon
Copy link
Member

sglyon commented Jan 29, 2018

Hmm, that is very strange.

Can you please post the code that you used when this error message comes about? I haven't seen it before, but perhaps with the exact sequence of commands I will be able to trigger the error also.

It would also be helpful to see the output of Pkg.status() so I know exactly what versions of PlotlyJS.jl and DataFrames.jl you are working with

@zhouyan
Copy link
Author

zhouyan commented Jan 29, 2018 via email

@sglyon
Copy link
Member

sglyon commented Jan 29, 2018

Ahh shoot, I actually see what the issue is...

The docstring here has a $(mean(df[:X1]), where the $ actually makes Julia try to run that code.

I've fixed this on master. I will do a quick release so that issue goes away on the official released version of the package

@sglyon
Copy link
Member

sglyon commented Jan 29, 2018

I'm actually working on a larger set of updates right now that I won't be able to finish really quickly.

For now, will you please try Pkg.checkout("PlotlyJS") and try again?

@zhouyan
Copy link
Author

zhouyan commented Jan 31, 2018

Sorry for late reply. After checkout the master it works correctly as far as I can tell.

Thanks

@sglyon
Copy link
Member

sglyon commented Feb 2, 2018

This is now fixed in a released version of PlotlyJS.jl

a Pkg.checkout("PlotlyJS") and Pkg.update() should get you back on that version

@sglyon sglyon closed this as completed Feb 2, 2018
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