-
Notifications
You must be signed in to change notification settings - Fork 21
Dataframe class name #17
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
Comments
I personally think that in Python there is some consistency in using lowercase capitalization for types and data structures: But if @devin-petersohn is ok with |
I am only okay with |
+1 to matching what the array / Array API standard does.
…On Thu, Jun 25, 2020 at 4:01 PM Devin Petersohn ***@***.***> wrote:
I am only okay with DataFrame if Array (capitalization) is how we are
spelling the array protocol. As long as we are consistent between the two,
I am okay.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#17 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAKAOIX4Z62EAHYWX2DEYF3RYO3J5ANCNFSM4OFQR62Q>
.
|
we didn't specify a name there - there's just "an array object". Reason: not needed (there's no way to call Status quo is |
I think there is consensus (correct me if I'm wrong), on having a 2-D structure where (at least) columns are labelled, and where a whole column share a type. More specific discussions about this structure can be made in #2.
In this issue, I'd like to discuss how we should name the class representing this structure. We've been using dataframe for the concept so far, and it's how the class is named in pandas, vaex, Modin, R and others. But in #14 (comment) it was proposed that we consider other names. I list here the proposed options in the comment and couple more. I propose that people write their username next to their preferred option, and use the comments to expand on why if needed.
Also, I think we should decide about capitalization, I guess these are the only options (using dataframe as example, but applied to the preferred option from the above list):
int
,datetime.datetime
,numpy.array
)The text was updated successfully, but these errors were encountered: