search

tcstewar / nstbot

Python library for controlling and emulating robots from the Neuroscientific System Theory at TUM
GNU General Public License v2.0
5 stars 9 forks source link

Optimized quick plotting #1

Closed studywolf closed 8 years ago

studywolf commented 9 years ago

instead of redrawing the whole image only update those parts that have changed, based on code from http://bastibe.de/2013-05-30-speeding-up-matplotlib.html

tcstewar commented 9 years ago

Cool. If this was done as an alternate display mode (i.e. you have to set display_mode='ubuntu_quick' for this to do anything, otherwise it'd fall back to the original version) then I'd be fine to merge it now. As it is, though, I'd need to test it on a bunch of different systems to see how that functions on them. which I'll hopefully get around to after the summer school.

studywolf commented 9 years ago

done!

studywolf commented 9 years ago

bump!

tcstewar commented 9 years ago

with the one typo I noted above, this runs fine for me on windows. Has it been tested on OS X?

tcstewar commented 9 years ago

Has it been tested on OS X?

(note: I'm not too worried about making sure it's been tested on OS X: I'm pretty sure I never tested the code on it originally. I'm just curious if it runs... :) )

studywolf commented 9 years ago

that's a very good question! i avoid macs like the plague so I'm not sure!

change made! and within a month this time!

studywolf commented 8 years ago

bump!

studywolf commented 8 years ago

bump!

tcstewar commented 8 years ago

bump!

Oops.. yes, I should really do this. I just need to find a camera to test it on.... it looks like there's one pointing at the robot arm that I probably shouldn't disturb -- do you know where the other one is?

studywolf commented 8 years ago

you can grab either of those ones! it's fine :)

On Fri, Mar 18, 2016 at 1:17 PM, tcstewar notifications@github.com wrote:

bump!

Oops.. yes, I should really do this. I just need to find a camera to test it on.... it looks like there's one pointing at the robot arm that I probably shouldn't disturb -- do you know where the other one is?

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/tcstewar/nstbot/pull/1#issuecomment-198459684

tcstewar commented 8 years ago

okay, I've confirmed this still works under windows, and I assume you're happy with how it works in Ubuntu, so I'll go ahead and merge... :)

studywolf commented 8 years ago

success!!

On Fri, Mar 18, 2016 at 1:34 PM, tcstewar notifications@github.com wrote:

Merged #1 https://github.com/tcstewar/nstbot/pull/1.

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/tcstewar/nstbot/pull/1#event-595226679