Add optional utilities for conversion to a DataFrame and for plotting with Gadfly #5
Conversation
with Gadfly. Note that this adds a dependency to the Requires package.
…o dataframe Conflicts: src/AxisArrays.jl
|
Yes, now the real fun can begin! Good call on I think we can have separate test REQUIRES file, so that'd be something to look into eventually (but isn't really critical for now). |
Add optional utilities for conversion to a DataFrame and for plotting with Gadfly
|
julia> using AxisArrays
WARNING: Method definition require(Symbol) in module Base at loading.jl:317 overwritten in module AxisArrays at /home/tim/.julia/v0.5/Requires/src/require.jl:12.I'd like to recommend that we follow @tkelman's advice. Can this functionality be moved to a separate package? |
|
Yup, I fully support that. |
|
@tshort, are you OK with that? If so, do you want to host the package? Or put it at JuliaStats? I don't have commit access there, so I couldn't host it. If needed I'll start a package and request that it be transferred to a suitable home. |
|
I'd prefer just deleting/disabling the code and waiting until we get the extern keyword (JuliaLang/julia#15705). |
|
I agree, the world is waiting eagerly for OK, I can do that. Thanks to git it's easy to get back. Thanks for your flexibility! |
|
For the record, commit 08f266d has |
This adds a dependency on Requires.jl. It's a small package that has worked well for me. The main advantage of this code is that it gives us a quick way to plot AxisArrays using Gadfly.
Note that this also requires the development version of DataFrames (I added a feature needed for this).
Note that this causes a ton of warnings when DataFrames and Gadfly are loaded. Most are due to other packages, but some are ambiguity warnings between DataArrays and AxisArrays. DataArrays will probably undergo a lot of change as they convert to some sort of Nullables representation, so I wouldn't worry about these for now. Ambiguity warnings are a PITA. Hopefully, Jeff's type work will fix all that.