Skip to content

Clean subplots #2227

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.

Sign up for GitHub

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

Merged
merged 17 commits into from
Jan 3, 2018
Merged

Clean subplots #2227

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
687a2e2
refactor calcdata filtering
alexcjohnson Nov 3, 2017
9a18062
perf xaCheater/xaNonCheater test in cartesian axes defaults
alexcjohnson Nov 4, 2017
2c9d90f
lint cartesian layout_defaults
alexcjohnson Nov 4, 2017
3625be4
rework subplot logic - create subplots from traces and components only
alexcjohnson Nov 22, 2017
2de217e
update baselines for improved subplot logic
alexcjohnson Dec 29, 2017
b94ad8e
wider acceptance range for png toimage test
alexcjohnson Dec 29, 2017
c88e8c2
update trans keys
alexcjohnson Dec 22, 2017
26aaffc
fix typo in cameraposition -> camera cleaning
alexcjohnson Nov 30, 2017
276d1d3
updated gl3d "cameraposition" baselines
alexcjohnson Dec 29, 2017
c49007e
:hocho: unused import
alexcjohnson Dec 29, 2017
698561c
:hocho: commented out code
alexcjohnson Dec 30, 2017
8804311
Object.keys -> for in, in plots/plots
alexcjohnson Dec 30, 2017
4786b6e
simplify redundant title removal
alexcjohnson Dec 30, 2017
6e62b35
get rid of unneeded _has(gl2d)
alexcjohnson Dec 30, 2017
0506f88
use _subplots in Axes.list/listIds
alexcjohnson Dec 30, 2017
2aa1545
simplify and speed up a few Axes.list uses
alexcjohnson Dec 30, 2017
44060db
fix annotation calc_autorange change
alexcjohnson Dec 30, 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
2 changes: 1 addition & 1 deletion test/jasmine/tests/toimage_test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ describe('Plotly.toImage', function() {
.then(function() { return Plotly.toImage(gd, {format: 'png', imageDataOnly: true}); })
.then(function(d) {
expect(d.indexOf('data:image/')).toBe(-1);
expect(d.length).toBeWithin(53660, 1e3, 'png image length');
expect(d.length).toBeWithin(54660, 3e3, 'png image length');
Copy link
Collaborator Author

@alexcjohnson alexcjohnson Dec 29, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This had been failing on my computer for ages, though it always works on CI... now there's just one test, in gl3d code, that consistently fails for me, though I need to be careful to have the chrome window on my secondary monitor; if I let it live on my laptop screen (retina) a whole bunch of other tests fail too. Must be a devicePixelRatio dependence or something. Would love to sort this out at some point, makes it annoying to run the full test suite if I'm not at home, and god forbid I ever upgrade this monitor and only have high-res devices to work with!

})
.then(function() { return Plotly.toImage(gd, {format: 'jpeg', imageDataOnly: true}); })
.then(function(d) {
Expand Down