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 controls to the heatmap example. #577

Merged
merged 2 commits into from
May 16, 2016
Merged

Conversation

manthey
Copy link
Contributor

@manthey manthey commented May 13, 2016

Move the example data set to data.kitware.com.

Allow selecting the cities and earthquake data sets.

Fix an issue with updating the radius or blurRadius on the fly (the circle wasn't being recomputed).

Fix an issue with changing the color gradient on the fly (the gradient wasn't being recomputed).

Move the example data set to data.kitware.com.

Allow selecting the cities and earthquake data sets.

Fix an issue with updating the radius or blurRadius on the fly (the circle wasn't being recomputed).

Fix an issue with changing the color gradient on the fly (the gradient wasn't being recomputed).
@codecov-io
Copy link

codecov-io commented May 13, 2016

Current coverage is 72.79%

Merging #577 into master will increase coverage by <.01%

@@             master       #577   diff @@
==========================================
  Files            82         82          
  Lines          7128       7131     +3   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           5188       5191     +3   
  Misses         1940       1940          
  Partials          0          0          

Powered by Codecov. Last updated by b3e15f8...1ac2c32

@aashish24
Copy link
Member

LGTM 👍

@manthey manthey merged commit 6024c4a into master May 16, 2016
@manthey manthey deleted the heatmap-example-controls branch May 16, 2016 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants