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

Kiva explorer demo #246

Merged
merged 3 commits into from
Oct 26, 2016
Merged

Kiva explorer demo #246

merged 3 commits into from
Oct 26, 2016

Conversation

corranwebster
Copy link
Contributor

Add a really simple traitsui view for playing with Kiva.

screen shot 2016-10-26 at 9 30 24 am

@codecov-io
Copy link

codecov-io commented Oct 26, 2016

Current coverage is 52.65% (diff: 100%)

Merging #246 into master will increase coverage by 0.65%

@@             master       #246   diff @@
==========================================
  Files           126        126          
  Lines         13190      13521   +331   
  Methods           0          0          
  Messages          0          0          
  Branches       1948       2068   +120   
==========================================
+ Hits           6859       7120   +261   
- Misses         5885       5932    +47   
- Partials        446        469    +23   

Powered by Codecov. Last update f35d9da...b32bac2

@jwiggins
Copy link
Member

😍

Bless you, sir. This is exactly what I need right now.

I should probably push my pyagg backend branch, huh?

@jwiggins
Copy link
Member

Just a quick read from my end, but LGTM

@corranwebster
Copy link
Contributor Author

You're welcome! And since this is non-critical code, I shall merge it.

@corranwebster corranwebster merged commit 3e1594b into master Oct 26, 2016
@corranwebster corranwebster deleted the demo/kiva-explorer branch October 26, 2016 14:11
@eric-jones
Copy link
Member

It would be useful to have something that printed out the rendering time on this... (per render and frames per second)

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.

4 participants