small bug in set_axes_limits

[martalmeida]

There is a small bug in set_axes_limits. The argument ax should be used in the self.drawmapboundary calls, here:

    # if map boundary not yet drawn for elliptical maps, draw it with default values.
    if not self._mapboundarydrawn or self._mapboundarydrawn not in ax.patches:
        # elliptical map, draw boundary manually.
        if (self.projection in ['ortho','geos','nsper','aeqd'] and
            self._fulldisk) or self.round or self.projection in _pseudocyl:
            # first draw boundary, no fill
            limb1 = self.drawmapboundary(fill_color='none')  <----------------|
            # draw another filled patch, with no boundary.
            limb2 = self.drawmapboundary(linewidth=0)        <----------------|
            self._mapboundarydrawn = limb2

I found this cos I was getting a funny ellipse on my colorbar!

[WeatherGod]

Hmm, indeed. That seems like an easy enough fix. I'll put together a PR for
that.

On Thu, Jul 16, 2015 at 12:14 PM, M Marta Almeida notifications@github.com
wrote:

There is a small bug in set_axes_limits. The argument ax should be used in
the self.drawmapboundary calls, here:

# if map boundary not yet drawn for elliptical maps, draw it with default values.
if not self._mapboundarydrawn or self._mapboundarydrawn not in ax.patches:
    # elliptical map, draw boundary manually.
    if (self.projection in ['ortho','geos','nsper','aeqd'] and
        self._fulldisk) or self.round or self.projection in _pseudocyl:
        # first draw boundary, no fill
        limb1 = self.drawmapboundary(fill_color='none')  <----------------|
        # draw another filled patch, with no boundary.
        limb2 = self.drawmapboundary(linewidth=0)        <----------------|
        self._mapboundarydrawn = limb2

I found this cos I was getting a funny ellipse on my colorbar!

—
Reply to this email directly or view it on GitHub
#193.

[WeatherGod]

Just to note, PR #194 should fix this.