-
Notifications
You must be signed in to change notification settings - Fork 0
Without DataFrames #1
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 clickin 8000 g “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
Comments
I'll take a look. |
Just for a point of comparison, CSV.jl depends on the generic Tables.jl interface and lets users use whichever table structure they need. |
Like this? === StructArray Output === === NamedTuples Output === === Columnar Output === |
Yeah. |
|
I'm thinking like this |
My leaning would be for positional argument rather than keyword and a type rather than a symbol, so it's an extensible interface, rather than an internal pattern match. |
Send a pull request?
…On Tue, Apr 1, 2025 at 6:38 PM, jariji ***@***.***(mailto:On Tue, Apr 1, 2025 at 6:38 PM, jariji <<a href=)> wrote:
My leaning would be for positional argument rather than keyword and a type rather than a symbol, so it's an extensible interface, rather than an internal pattern match.
—
Reply to this email directly, [view it on GitHub](#1 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AAASHGD4OPDZX4YY4SQJSBT2XM5RZAVCNFSM6AAAAAB2GCPLRCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONZRGA2DCNBRHA).
You are receiving this because you commented.Message ID: ***@***.***>
[jariji] jariji left a comment [(technocrat/ACS.jl#1)](#1 (comment))
My leaning would be for positional argument rather than keyword and a type rather than a symbol, so it's an extensible interface, rather than an internal pattern match.
—
Reply to this email directly, [view it on GitHub](#1 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AAASHGD4OPDZX4YY4SQJSBT2XM5RZAVCNFSM6AAAAAB2GCPLRCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONZRGA2DCNBRHA).
You are receiving this because you commented.Message ID: ***@***.***>
|
I'd like an option to use this without DataFrames.jl, using other Tables.jl table types like StructArrays.jl or
Vector{NamedTuple}
orNamedTuple{Vector}
. For example,get_acs(StructArray, ...)
.Then I wouldn't have to depend on or import DataFrames.jl and could directly use whatever table I want instead of needing to convert to it. DataFrames.jl could be supported in an extension package but wouldn't need to be a direct dependency of ACS.jl.
The text was updated successfully, but these errors were encountered: