-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Regl-based scattergl and scatterpolargl traces #2258
Merged
Merged
Changes from 1 commit
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
b64928d
regl-ify scattergl by dfcreative
etpinard 70769e9
Merge branch 'master' into rescatter-squashed
etpinard 45f9003
Merge branch 'rescatter-squashed-2018' into regl-scattergl-scatterpol…
etpinard a6f86db
first steps for scatterpolargl
etpinard 0983b21
Make toSVG
dy 2b5ee19
Handle multiple subplots selection properly
dy f235b0d
Fix single-color open markers
dy 58cb8f4
Make proper fill behaviour
dy ab2a0a2
Fix gl2d_click & gl_plot_interact tests
dy 0842891
reorder gl2d <-> gl3d tests
etpinard ed075d2
skip gl2d visibility toggle test on CI
etpinard 69a033b
more tolerance gl2d annotation test
etpinard 9f304e6
sort pointcloud and heatmapgl mock
etpinard 3fbf523
update gl2d baseline images
etpinard 1d650ac
Bump regl
dy 91d4a75
Add multiple axes test mocks
dy dce0a1a
Normalize marker sizes
dy 9f9f512
add mocks to lock scattergl fixups:
dy 9911ba8
Fix log error bars
dy dc00cf1
Add gl2d layout images and shapes mock
dy 9548cc1
add long tick labels mocks
dy 65bd43d
ADd scatter panel test/fix
dy 10f87a3
Add gl2d line dash test
dy 715683f
fixup gl2d zoombox tests
etpinard b45e91d
try re-balancing tests in the 2 containers to reduce exec time
etpinard c0c0189
cleanup scattergl index exports
etpinard 1075d87
make scatterpolar use own attributes and defaults
etpinard ebaf705
fix select/lasso click event + test
etpinard ac7db75
rm unsed dependencies and modules
etpinard 8e3a0ba
make Plots.generalUpdatePerTraceModule consistent w/ Cartesian.plot
etpinard e6223b6
:palm_tree: scatterpolar <-> scatterpolargl hoverPoints
etpinard 5ea57a3
Fix gl marker symbols logic/test
dy d3fd7d2
Add fill self next test
dy f48cc2f
Fix regl-error2d update
dy de7bea3
baselines for dima's last few commits
etpinard a237c83
add some scatterpolargl tests
etpinard 44b0ae6
make gl2d drag test use assets/drag
etpinard 40f60de
attempt for making polar drag test more robust
etpinard 8bebc2e
skip polar drag test on CI (for now)
etpinard File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what does this fix?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see it fixes #2259 (comment)
Thanks.