Summary: | Adding curves through js is slow | ||
---|---|---|---|
Product: | [Applications] kst | Reporter: | Andrew Walker <arwalker> |
Component: | general | Assignee: | kst |
Status: | RESOLVED FIXED | ||
Severity: | wishlist | CC: | bug.zilla.vynce |
Priority: | NOR | ||
Version: | 1.8.0 | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
Andrew Walker
2010-01-27 20:08:11 UTC
SVN commit 1081190 by arwalker: CCBUG:224516 first draft. Add enableUpdates property on the Kst js-object. This will allow prevention of automatic repaints with each curve addition. When all curves are added call Kst.enableUpdates=true M +5 -5 extensions/js/bind_arrow.cpp M +19 -19 extensions/js/bind_axis.cpp M +3 -3 extensions/js/bind_axislabel.cpp M +3 -3 extensions/js/bind_axisticklabel.cpp M +4 -4 extensions/js/bind_borderedviewobject.cpp M +5 -5 extensions/js/bind_box.cpp M +8 -8 extensions/js/bind_curvecollection.cpp M +3 -3 extensions/js/bind_ellipse.cpp M +4 -4 extensions/js/bind_group.cpp M +30 -0 extensions/js/bind_kst.cpp M +15 -0 extensions/js/bind_kst.h M +8 -8 extensions/js/bind_label.cpp M +9 -9 extensions/js/bind_legend.cpp M +6 -6 extensions/js/bind_line.cpp M +3 -3 extensions/js/bind_picture.cpp M +2 -2 extensions/js/bind_plot.cpp M +5 -5 extensions/js/bind_plotlabel.cpp M +3 -3 extensions/js/bind_timeinterpretation.cpp M +8 -8 extensions/js/bind_viewobject.cpp M +5 -5 extensions/js/bind_viewobjectcollection.cpp M +22 -9 libkstapp/kst.cpp M +5 -0 libkstapp/kst.h WebSVN link: http://websvn.kde.org/?view=rev&revision=1081190 The previous message should read enableRepaints in place of enableUpdates. Some example code: . Kst.enableRepaints=false . <add many curves to a plot> . Kst.enableRepaints=true . Changing the Kst.enableRepaints property doesn't seem to work. From the JS console: kst> Kst.enableRepaints=false false kst> Kst.enableRepaints true Painting also still occurs. SVN commit 1081640 by arwalker: CCBUG:224516 add support for setting properties on the Kst object M +18 -0 bind_kst.cpp M +1 -0 bind_kst.h WebSVN link: http://websvn.kde.org/?view=rev&revision=1081640 Hi Michael, My apologies for that. The Kst object didn't have underlying support for setting the values of the properties, as they were previosuly all read-only. I've added the necessary support and checked-in the change. I'll mark the bug as fixed once we've done some testing at this end. Thanks, Andrew ----- Original Message ----- From: "Michael Vincent" <bug.zilla.vynce@neverbox.com> To: <arwalker@sumusltd.com> Sent: Thursday, January 28, 2010 1:20 PM Subject: [Bug 224516] Adding curves through js is slow > https://bugs.kde.org/show_bug.cgi?id=224516 > > > Michael Vincent <bug.zilla.vynce@neverbox.com> changed: > > What |Removed |Added > ---------------------------------------------------------------------------- > CC| > |bug.zilla.vynce@neverbox.co > | |m > > > > > --- Comment #3 from Michael Vincent <bug zilla vynce neverbox com> > 2010-01-28 22:20:34 --- > Changing the Kst.enableRepaints property doesn't seem to work. > > From the JS console: > kst> Kst.enableRepaints=false > false > kst> Kst.enableRepaints > true > > Painting also still occurs. > > -- > Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email > ------- You are receiving this mail because: ------- > You reported the bug. > Testing shows that things are now working as expected. It works now. Thanks! |