Summary: | observing list wizard is extremely slow | ||
---|---|---|---|
Product: | [Applications] kstars | Reporter: | Olivier Vitrat <ovit.debian> |
Component: | general | Assignee: | Akarsh Simha <akarsh.simha> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | akarsh.simha |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Debian testing | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: | |||
Attachments: | Improve usability by making it optional to update the object count |
Description
Olivier Vitrat
2007-03-11 14:57:52 UTC
Confirmed in KDE4 versin as well. I'll need to figure out a way to keep the GUI responsive and let the user know that progress is being made. I've identified the problem. We are making a call to a rather expensive function several thousand times, unnecessarily. Will try to fix it shortly. Created attachment 29418 [details]
Improve usability by making it optional to update the object count
The patch avoids updating the object count every time it changes, but instead provides an "Update Count" button and updates the count only when the button is clicked.
There seems to be no clean way of speeding up the algorithm in all contexts. It should be possible, but I think it will require a lot of coding effort and time. The patch I attached earlier works around the problem by instead making it the user's option to update the count of objects. However, we can't commit this patch now because it'll break the translated-string freeze, so it won't be fixed in KDE 4.2. Will commit it after the freeze is lifted. SVN commit 909005 by asimha: Improve usability of the Observing List Wizard by making the count update optional. BUG: 142833 CCMAIL: kstars-devel@kde.org M +24 -12 obslistwizard.cpp M +1 -0 obslistwizard.h M +15 -0 obslistwizard.ui WebSVN link: http://websvn.kde.org/?view=rev&revision=909005 |