-
Notifications
You must be signed in to change notification settings - Fork 2
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
performance of widget #6
Comments
Hi! There are many performance issues bugging me as well. I'm not very familiar with the %notebook backend for matplotlib, but I think that this is coming from the fact that by default it shows the pointer position as (x,y) double on the bottom of the plots, so it's redrawing everytime. We can look into this, but I would not wait for this for the release. let's leave this open |
Maybe one can disable this "show position" feature. |
Was looking into that this morning. Matplotlib documentation on the
%notebook backend is scarce. There are other "interactive" backends that
we can try out (tried some with much sucess).
More pressing issues right now, but still, this is important, thanks for
pointing it out.
…On 09.03.2017 14:14, Martin K. Scherer wrote:
Maybe one can disable this "show position" feature.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AHK3NJu595a1QVO-p3cZvQWqvxqTDYuuks5rj_tTgaJpZM4MXc2Y>.
--
Dr. Guillermo Pérez-Hernández
Freie Universität Berlin
Institute for Mathematics
Arnimallee 6
D-14195 Berlin
tel 0049 30 838 75775
http://userpage.fu-berlin.de/gph82/
http://orcid.org/0000-0002-9287-8704
--
|
I just noticed, that the callbacks for updating the widgets are executed for every movement of the mouse in the free energy landscape plot, no matter if the position is actually committed by click. This leads to degraded performance (maybe I should buy new hardware too).
It should be possible to update only if something is really changed and we need to recompute/redraw.
The text was updated successfully, but these errors were encountered: