Skip to content

pcolormesh edgecolor of "None" #999

Closed

Description

@pelson

#901 implemented a direct check that the edgecolor was not "None", but this excludes the use of "none" (or any other case derivative):

import numpy
import matplotlib.pyplot as plt

plt.subplot(121)
plt.pcolormesh(numpy.arange(12).reshape(3, 4), edgecolors='none')
plt.subplot(122)
plt.pcolormesh(numpy.arange(12).reshape(3, 4), edgecolors='None')
plt.show()

Please consider implementing case insensitive equivalent behaviour.

Activity

  1. mdboom commented on Jul 10, 2012

    @mdboom
    Member

    I don't think #901 introduced this -- it removed a check (which was case sensitive) and allowed it to pass through to another part of the library (which is also case sensitive). I'm not sure I consider this a bug -- most things in matplotlib are case sensitive.

  2. efiring commented on Jul 15, 2012

    @efiring
    Member

    The relevant code appears to be in Collection.set_edgecolor, which converts everything to lower case. Although in general I prefer case-sensitivity, there are exceptions, and this is one; it is just too hard to remember that something should be "None", and something else should be "none". So, as far as I can see, things are working in a reasonable fashion.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions