Skip to content

Commit ecd0a27

Browse files
Fix heatmaps ignoring ColorLimits
1 parent 3bbc550 commit ecd0a27

File tree

2 files changed

+20
-0
lines changed

2 files changed

+20
-0
lines changed

plotly/Test_plotlyfig.m

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1145,6 +1145,8 @@ function testHeatmapData(tc)
11451145
x = {num2cell(num2str((1:5)'))}, ...
11461146
y = {num2cell(num2str(flip(1:5)'))}, ...
11471147
z = flip(data), ...
1148+
zmin = min(data,[],"all"), ...
1149+
zmax = max(data,[],"all"), ...
11481150
connectgaps = false, ...
11491151
hoverongaps = false, ...
11501152
hoverinfo = "text", ...
@@ -1173,6 +1175,20 @@ function testHeatmapData(tc)
11731175
), AbsTol=1e-15);
11741176
end
11751177

1178+
function testHeatmapDataColorLimitsOverride(tc)
1179+
fig = figure("Visible","off");
1180+
data = magic(5);
1181+
zmin = -5;
1182+
zmax = 5;
1183+
h = heatmap(data);
1184+
h.ColorLimits = [zmin, zmax];
1185+
1186+
p = plotlyfig(fig,"visible","off");
1187+
1188+
tc.verifyEqual(p.data{1}.zmin, zmin);
1189+
tc.verifyEqual(p.data{1}.zmax, zmax);
1190+
end
1191+
11761192
function testErrorbarData(tc)
11771193
fig = figure("Visible","off");
11781194
x = 1:10;
@@ -2470,6 +2486,8 @@ function testTiledLayoutHeatmap(tc)
24702486
x = {num2cell(num2str((1:2)'))}, ...
24712487
y = {num2cell(num2str(flip(1:2)'))}, ...
24722488
z = flip(data), ...
2489+
zmin = min(data,[],"all"), ...
2490+
zmax = max(data,[],"all"), ...
24732491
connectgaps = false, ...
24742492
hoverongaps = false, ...
24752493
hoverinfo = "text", ...

plotly/plotlyfig_aux/handlegraphics/updateHeatmap.m

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,8 @@
1414
data.x = heat_data.XDisplayData;
1515
data.y = heat_data.YDisplayData(end:-1:1, :);
1616
data.z = cdata;
17+
data.zmin = heat_data.ColorLimits(1);
18+
data.zmax = heat_data.ColorLimits(2);
1719
data.connectgaps = false;
1820
data.hoverongaps = false;
1921

0 commit comments

Comments
 (0)