Skip to content

Check Table MaskedColumn for future change in MaskedArray #5530

Closed

Description

@mhvk

In numpy 1.13, MaskedArray views will start to share the mask with the item viewed rather than get a new copy (numpy/numpy#5580). In many ways, this is much more logical, and I think it should pose no issues for MaskedColumn, but we should check. In particular, we should ensure that in any copy, the mask actually does get copied.

Activity

  1. added this to the Future milestone on Dec 1, 2016
  2. mhvk commented on Dec 1, 2016

    @mhvk
    ContributorAuthor

    See #2997 (comment) for the types of warnings one may get from numpy while the change in behaviour is still being prepared

  3. mhvk commented on Dec 1, 2016

    @mhvk
    ContributorAuthor

    Actually, as pointed out by @saimn, there was already an issue for this: #5060. So, I'll close this here.

  4. bsipocz commented on Dec 1, 2016

    @bsipocz
    Member

    Maybe add the duplicate label, too?

  5. mhvk commented on Dec 1, 2016

    @mhvk
    ContributorAuthor

    Done.

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

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions