Skip to content

Consider moving all column attributes to info #7394

Open

Description

@mhvk

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.

Activity

  1. taldcroft commented on Apr 19, 2018

    @taldcroft
    Member

    @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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions