Parts of the code are unnecessary and slow

Bug #408476 reported by Umang Varma
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Pynagram
Invalid
Undecided
Unassigned

Bug Description

Some parts of the back-end and GUI are slow and perform some tasks more often than required (e.g. updating the labels). This reduces performance and code standards.
Some examples:

 * App.__update_words is used too often (often not required)
 * anagram.guess doesn't return whether the word has already been typed in or not. Once the anagram class starts tracking score also, then this will be check twice instead of once

More such issues should, of course, be weeded out before 0.1.2, since they are relatively trivial cases.

Any changes in the back-end should be reflected in the unit tests!

Umang Varma (umang)
description: updated
Umang Varma (umang)
summary: - Parts of the code are unneccsary and slow
+ Parts of the code are unnecessary and slow
Revision history for this message
Umang Varma (umang) wrote :

Though the two mentioned originally have actually been fixed, this bug with the impression that a lot could be done. After actually using timeit to see if there is any difference, it turns out that there is no visible difference.

Maybe I should have listened to Mark Pilgrim (http://www.diveintopython.org/performance_tuning/summary.html)!

Changed in pynagram:
assignee: Umang (umang) → nobody
importance: Medium → Undecided
milestone: 0.1.2 → none
status: Confirmed → Invalid
Revision history for this message
Umang Varma (umang) wrote :

(typo)
*this bug was filed with the impression

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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