Skip to content

heatmapgl in main bundle #1197

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

Merged
merged 6 commits into from
Nov 29, 2016
Merged

heatmapgl in main bundle #1197

merged 6 commits into from
Nov 29, 2016

Conversation

etpinard
Copy link
Contributor

@etpinard etpinard commented Nov 24, 2016

resolves #1080

This PR requires gl-vis/gl-heatmap2d#2 to be merged and published. Big ups @dfcreative !

In brief, while heamapgl was added as beta in 1.10.0 it wasn't up to plotly.js standards. With these changes (mainly in gl-vis/gl-heatmap2d#2 ), picking is vastly improved, image generation is now repeatable with no performance cost. That is, heatmapgl is capable of plotting 1e6 points in about 1 second.

To note, heatmapgl isn't completely on-par with heatmap. Mainly,

  • it has only one coloring mode (i.e. no zsmooth) roughly equivalent to zsmooth: false. That is, heatmap graphs do not display bricks! It is meant for large datasets.
  • has no connectgaps (this would be a great addition when we get the time)
  • has no xgap / ygap attributes (no need for those as heatmapgl doesn't support brick coloring)

- so that it registers contourgl
- instead of simply copying attribute from 'heatmap'
- attribute like 'zsmooth', 'xgap' and 'connectgaps'
  aren't available in 'heatmapgl'
@etpinard etpinard added this to the v1.21.0 milestone Nov 24, 2016
@etpinard etpinard changed the title Heatmapgl primetime heatmapgl in main bundle Nov 24, 2016
@etpinard etpinard merged commit 4d243a9 into master Nov 29, 2016
@etpinard etpinard deleted the heatmapgl-primetime branch November 29, 2016 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature something new
Projects
None yet
Development

Successfully merging this pull request may close these issues.

heatmapgl TODO list
1 participant