GTG

Update boot-up performance option

Bug #897438 reported by Izidor Matušov
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
GTG
Fix Released
Critical
Izidor Matušov
GTG 0.3 "make it stable"

Bug Description

Instead of running it only through a single iteration, it should wait until all tasks are loaded and after that immediately finish. The change is needed because of the change of LibLarch.

Solving this bug may facilitate tackling the performance issue on load.

Do not forget to update manpage documentation.

Related branches

lp:~izidor/gtg/performance
Bertrand Rousseau (community): Approve
Diff: 319 lines (+59/-62)
10 files modified
CHANGELOG (+1/-0)
GTG/core/datastore.py (+2/-15)
GTG/core/plugins/api.py (+6/-7)
GTG/gtk/browser/browser.py (+5/-1)
GTG/gtk/browser/treeview_factory.py (+11/-3)
GTG/plugins/urgency_color/urgency_color.py (+1/-2)
GTG/tools/import_liblarch.py (+11/-5)
GTG/tools/synchronized.py (+0/-14)
gtg (+4/-1)
scripts/debug.sh (+18/-14)
Revision history for this message
Lionel Dricot (ploum-deactivatedaccount) wrote :

I don't understand what you mean here. What should be done to fix this bug?

Revision history for this message
Izidor Matušov (izidor) wrote :

The current implementation of -b is to instead of gtk.main() run gtk.main_iteration(). Before LibLarch it means that GTG finished when the first idle situation occurred (everything was loaded)

Because of the change of LibLarch (it is asynchronous and process requests during those idle states), the implementation does not make sense and must be changed.

I believe that the feature was implemented because of profiling boot-up performance -- when everything was loaded, GTG finished and the measurement was not influenced by human factor (Is it already finished or not yet?)

Reworking this feature can help during performance optimization.

Izidor Matušov (izidor)
Changed in gtg:
assignee: nobody → Izidor Matušov (izidor)
status: Confirmed → In Progress
Revision history for this message
Izidor Matušov (izidor) wrote :

Merged at rev 1177

Changed in gtg:
status: In Progress → Fix Committed
Bertrand Rousseau (bertrand-rousseau)
Changed in gtg:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.