Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

Register XSV #1744

Merged
merged 1 commit into from
Nov 17, 2014
Merged

Register XSV #1744

merged 1 commit into from
Nov 17, 2014

Conversation

benhamner
Copy link
Contributor

No description provided.

@milktrader
Copy link
Contributor

This looks interesting. Would you consider an MIT license? I'm not clear on what other packages depending on XSV.jl would need to do to comply with the current copyright.

@IainNZ
Copy link
Member

IainNZ commented Nov 17, 2014

@milktrader this is 3-clause BSD, about the same as MIT in terms of permissiveness (I think)

IainNZ added a commit that referenced this pull request Nov 17, 2014
@IainNZ IainNZ merged commit 3b370df into JuliaLang:metadata-v2 Nov 17, 2014
@IainNZ
Copy link
Member

IainNZ commented Nov 17, 2014

Looking forward to a tagged version @benhamner , streaming CSV reading has been a requested feature in Base in the past

@johnmyleswhite
Copy link
Member

It's kind of amazing that you just published this, Ben: I'm about five days away from releasing my own general purpose CSV parsing library. Will be fun to compare.

@milktrader
Copy link
Contributor

Ok, thanks @IainNZ. So there are no special notices that need to be used to include this in a Julia package I presume?

This package may solve an outstanding issue in Quandl.jl (milktrader/Quandl.jl#1)

@benhamner
Copy link
Contributor Author

@johnmyleswhite cool! I just wrote this as a short Sunday afternoon project, after going down a rabbit hole of CSV parsing issues in Python and being frustrated by not having a good Julia option

Curious to see what your implementation entails - this is incredibly bareboned

@johnmyleswhite
Copy link
Member

Mine is much, much heavier: it's more like a CSV reader toolkit that allows you to implement an interface, which in turn will let you read data from CSV files into an arbitrary data structure. It's not really meant to be end-user facing, so the two libraries complement each other pretty well.

@StefanKarpinski
Copy link
Sponsor Member

I believe that 3-clause BSD is more restrictive than MIT; 2-clause BSD is closer to MIT.

@benhamner
Copy link
Contributor Author

I'll switch it to the MIT license on the next published version

@milktrader
Copy link
Contributor

Cool, thanks @benhamner

This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants