You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Consider moving all column attributes to info #7394
Currently, much of the additional attributes on columns are accessed via info, which helps treat mixin columns equally to regular ones. This could be taken even further, by also moving things like pprint to info, so that the way columns represent themselves in a table becomes something info is used for as well, thus making the separation of regular and mixin columns much less of a concern for Table.
@taldcroft made an initial implementation of this in #6720, which had some issues for MaskedColumn; this issue is a reminder that it would in fact be good to do this, if someone has the time to work out the kinks.
@mhvk - yes, this is still something that should be done and I hope to get back to it. I don't remember the specific problems but this was a little intertwined with the whole performance thing and why the masked array needed to be finalized so many times (#6721). Unfortunately Chandra is keeping me pretty busy so any help would be welcome.
Activity
taldcroft commentedon Apr 19, 2018
@mhvk - yes, this is still something that should be done and I hope to get back to it. I don't remember the specific problems but this was a little intertwined with the whole performance thing and why the masked array needed to be finalized so many times (#6721). Unfortunately Chandra is keeping me pretty busy so any help would be welcome.