-
Notifications
You must be signed in to change notification settings - Fork 13
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
load might support a cache mechanism #6
Comments
Using DataDeps with this should be trivial. something like (not tested so maybe has typos)
In fact, |
I guess I am missing why this wants to actually be part of queryverse, or FileIO.jl Queryverses in built functionality to work directly on URL is just a convenience for when it is convenient (small data, or single use scripts). To be clear, |
Yes I think for DataDeps a doc effort is required into Queryverse to give it some highlight. |
But I think some parameters of |
With RemoteFiles.jl, it can be achieved using using Queryverse
using RemoteFiles
data = RemoteFile("https://raw.githubusercontent.com/davidanthoff/CSVFiles.jl/master/test/data.csv")
download(data)
df = load(path(data)) |> DataFrame but maybe a better API should be defined. julia> load(RemoteFile("https://raw.githubusercontent.com/davidanthoff/CSVFiles.jl/master/test/data.csv")) |> DataFrame
ERROR:
MethodError: no method matching load(::RemoteFiles.RemoteFile)
Closest candidates are:
load(::Union{AbstractString, IO}, ::Any...; options...) at /Users/femto/.julia/v0.6/FileIO/src/loadsave.jl:113
load(::FileIO.Formatted{F}, ::Any...; options...) where F at /Users/femto/.julia/v0.6/FileIO/src/loadsave.jl:167 maybe |
Hello,
http://www.david-anthoff.com/jl4ds/stable/fileio.html shows usage such as
maybe a cache mechanism (such as https://github.com/helgee/RemoteFiles.jl or https://github.com/oxinabox/DataDeps.jl ) should be integrated to Queryverse (or at least its usage be better documented)
A possible API could be
or
So file will be stored with
data.csv
as filename (by default) and inside a default directoryPinging @oxinabox and @helgee
But this is probably not the right place to suggest this... maybe FileIO?
Kind regards
The text was updated successfully, but these errors were encountered: