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

Eliminate CSV dependency #129

Open
PallHaraldsson opened this issue Dec 6, 2021 · 0 comments
Open

Eliminate CSV dependency #129

PallHaraldsson opened this issue Dec 6, 2021 · 0 comments

Comments

@PallHaraldsson
Copy link
Contributor

PallHaraldsson commented Dec 6, 2021

For this package all the CSV files are relatively small I think.

CSV is a heavy dependency, on startup. It pays off if you load large CSV files. I was thinking maybe you could rather use the really fast (built into Base), it seems even with compressed files:

julia> @time using DelimitedFiles
  0.004993 seconds (1.15 k allocations: 82.234 KiB, 78.42% compilation time)

https://docs.julialang.org/en/v1/stdlib/DelimitedFiles/

A downside could be: If in all real-world code you would load CSV anyway, then not effective (but neither slower this way). Another option could be, this package if often (not always) used with RCall (what I'm looking into now). And plausibly then at least you could rather use CSV reading from R than Julia's CSV. Or Python's tool where appropriate.

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

1 participant