-
Notifications
You must be signed in to change notification settings - Fork 50
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
Tables ArrayColumn docs 9957 #279
Tables ArrayColumn docs 9957 #279
Conversation
:param long size: The width of the array, `>= 1` | ||
|
||
:param [][] values: A list of arrays, each of length :attr:`size` | ||
(optional). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In what sense is it optional? I feel like the argument could omitted entirely but I don't see a corresponding constructor, so is it just that one can pass null or somesuch?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It can be omitted entirely (effectively defaults to None), at least in Python, not sure how it works in opther languages. The [ ] was meant to indicate an optional arg, unfortunately it doubles as a list indicator.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ohh, thanks! Yes, that makes more sense now.
@mtbc Hypenated |
Thank you for indulging me. (-: Good to merge. |
The belated second part of ticket 9957, documentation for ArrayColumns.
This applies to new features, so should be held back until the next release.