Skip to content

2D Normalization & Colormapping for ScalerMappables #8738

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 35 commits into from
Closed
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
201dc2a
Initailize class 2D norm
patniharshit Jun 9, 2017
7217536
Change name of class to BivariateNorm, take normalizer instances sepa…
patniharshit Jun 9, 2017
0178730
Bivariate works with imshow, pcolor, pcolormesh, pcolorfast
patniharshit Jul 3, 2017
575a244
add blank line between bivariate classes
patniharshit Jul 9, 2017
8425637
Add support for bivariate in scatter
patniharshit Jul 9, 2017
4a5831d
Fix missing norm2 in BivaraiteNorm
patniharshit Jul 11, 2017
85df736
Fixed bug in if condition bivariate mapping
patniharshit Jul 12, 2017
8e1bc63
Add autoscale, autoscale_None, scaled methods for BivariateNorm
patniharshit Jul 13, 2017
a9aace3
trying to show ticks and labels on both x and y axis
patniharshit Jul 16, 2017
55a7a68
commented outline code, changed gridspace to show sqaure colorbar
patniharshit Jul 17, 2017
c2eb617
2d colorbar printing now
patniharshit Jul 17, 2017
daef751
Add code for Colorsquare
patniharshit Jul 22, 2017
96af3d3
Modify axis aspect and fraction for colorsquare
patniharshit Jul 22, 2017
9285372
fix failing commits
patniharshit Jul 22, 2017
01b5932
Change PEP8 violations
patniharshit Jul 27, 2017
28288e2
Revert isBivari, remove classic locator
patniharshit Jul 29, 2017
ca2b111
Subclass normalizers from Abstract Base Class
patniharshit Aug 2, 2017
75b4eba
move _ticker function inside update_ticks
patniharshit Aug 2, 2017
bf65847
move _ticker function inside update_ticks
patniharshit Aug 2, 2017
f9dc4b1
remove cmap in imshow and pcolorfast
patniharshit Aug 5, 2017
8a985f0
Defer call to norm and cmap in imshow
patniharshit Aug 7, 2017
8f6928d
Fix bivariate handling with pcolormesh
patniharshit Aug 15, 2017
d54cb44
Make pcolor work with bivariate
patniharshit Aug 16, 2017
4d03552
Do 2D->1D mapping in call method of colormap
patniharshit Aug 20, 2017
2573610
Use list comprehensions to ravel
patniharshit Aug 20, 2017
9228480
Reflect on reviews
patniharshit Aug 21, 2017
5c8dc65
Update docstrings
patniharshit Aug 25, 2017
21ea65f
Add bivariate example
patniharshit Aug 25, 2017
b8d43a9
Treat one as slightly less than one
patniharshit Aug 25, 2017
7681d14
Change conditions for bivar
patniharshit Aug 25, 2017
59f56af
Add image comparison test for bivariate
patniharshit Aug 25, 2017
f98978f
Update is_bivar condition to expect either BivariateNorm or Bivariate…
patniharshit Aug 26, 2017
75a289f
Add line continuation in docstrings
patniharshit Aug 27, 2017
7803974
Debug doc build fail
patniharshit Aug 28, 2017
e40c87b
Add missing symbols in docs
patniharshit Aug 28, 2017
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add image comparison test for bivariate
  • Loading branch information
patniharshit committed Aug 27, 2017
commit 59f56af84f9b019ddcd3a6a06b35aa9c6521c548
3 changes: 2 additions & 1 deletion lib/matplotlib/axes/_axes.py
Original file line number Diff line number Diff line change
Expand Up @@ -5150,7 +5150,8 @@ def imshow(self, X, cmap=None, norm=None, aspect=None,
"'mcolors.BivariateNorm'")
raise ValueError(msg)

is_bivari = (X.ndim == 3 or X.shape[0] == 2)
temp = np.asarray(X)
is_bivari = (temp.ndim == 3 or temp.shape[0] == 2)
if is_bivari:
if cmap is None:
cmap = mcolors.BivariateColormap()
Expand Down
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
28 changes: 28 additions & 0 deletions lib/matplotlib/tests/test_axes.py
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@
from numpy.testing import assert_allclose, assert_array_equal
from matplotlib.cbook import IgnoredKeywordWarning
from matplotlib.cbook._backports import broadcast_to
from matplotlib.cbook import get_sample_data

# Note: Some test cases are run twice: once normally and once with labeled data
# These two must be defined in the same test function or need to have
Expand Down Expand Up @@ -5271,3 +5272,30 @@ def test_twinx_knows_limits():
def test_zero_linewidth():
# Check that setting a zero linewidth doesn't error
plt.plot([0, 1], [0, 1], ls='--', lw=0)


@image_comparison(
baseline_images=['bivar_imshow', 'bivar_pcolor', 'bivar_pcolormesh',
'bivar_pcolorfast'],
extensions=['png']
)
def test_bivariates():
air_temp = np.load(get_sample_data('air_temperature.npy'))
surf_pres = np.load(get_sample_data('surface_pressure.npy'))
bivariate = [air_temp, surf_pres]

fig1, ax1 = plt.subplots()
cax1 = ax1.imshow(bivariate)
cbar = fig1.colorbar(cax1, xlabel='air_temp', ylabel='surf_pres')

fig2, ax2 = plt.subplots()
cax2 = ax2.pcolor(bivariate)
cbar = fig2.colorbar(cax2, xlabel='air_temp', ylabel='surf_pres')

fig3, ax3 = plt.subplots()
cax3 = ax3.pcolormesh(bivariate)
cbar = fig3.colorbar(cax3, xlabel='air_temp', ylabel='surf_pres')

fig4, ax4 = plt.subplots()
cax4 = ax4.pcolorfast(bivariate)
cbar = fig4.colorbar(cax4, xlabel='air_temp', ylabel='surf_pres')