Skip to content
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

Add heatmapgl trace module #427

Merged
merged 6 commits into from
Apr 15, 2016
Merged

Add heatmapgl trace module #427

merged 6 commits into from
Apr 15, 2016

Conversation

etpinard
Copy link
Contributor

@etpinard etpinard commented Apr 14, 2016

@mdtusz @bpostlethwaite @mikolalysenko

image

as discussed, heatmapgl won't be part of the dist plotly.js bundle - until gl2d plots can have reliable image test support.

To register heatmapgl :

var Plotly = require('plotly.js/lib/core');

Plotly.register(
  require('plotly.js/lib/heatmapgl')
);

Plotly.plot(/* as normally */)

@mdtusz
Copy link
Contributor

mdtusz commented Apr 14, 2016

https://www.youtube.com/watch?v=G_f8ayQQx4I

@mdtusz
Copy link
Contributor

mdtusz commented Apr 15, 2016

💃

@etpinard etpinard merged commit 5c60dbe into master Apr 15, 2016
@etpinard etpinard deleted the gl-heatmap2d branch April 15, 2016 17:26
@hoidn
Copy link

hoidn commented May 8, 2016

Thank you for this very promising feature.

I was curious whether it would be straightforward for one to modify plolty.js and plolty.py to make it available from the python API? I rebuilt the plotly.js bundle after adding

require('./heatmapgl')

to the argument list for Core.register in lib/index.js. Now I suppose it's necessary to add heatmapgl to the json schema on the python end, but I don't know how to go about that.

@etpinard
Copy link
Contributor Author

etpinard commented May 9, 2016

@hoidn here's how I would go about running a custom plotly.js bundle in plotly.py: https://gist.github.com/etpinard/21eb47feaef0acfde93cea37c6a3710a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants