[Biopython-dev] [Bug 2494] _retrieve_taxon in BioSQL.py needs urgent optimization

[bugzilla-daemon at portal.open-bio.org]

http://bugzilla.open-bio.org/show_bug.cgi?id=2494





------- Comment #2 from biopython-bugzilla at maubp.freeserve.co.uk  2008-04-26 06:29 EST -------
There is a small risk that your numbers will be missleading when applied to
other databases (i.e. mysql versus postgres).

Other than that, using just the parent id is probably a much better idea
(especially given some of the proposals on Bug 2475 about not writing the
left/right values).

Have you ever used the "diff" command-line tool to produce a patch file?  e.g.
diff old_version.py new_version.py > patch.txt

Or, if you are working on a CVS checkout, modify the local file and then:
cvs diff changed_file.py > patch.txt

Also read up on the "patch" command for applying the patch to update an
unchanged file.  If you are on a Unix style platform, these are usually
installed already.  For Windows I use cygwin's diff command but there are
probably other options.

There are several advantages, including the fact that patches are smaller. For
initial code review, they also highlight the area changed.  Another big
advantage is if CVS has been updated in the meantime, a patch can often still
be applied automatically.


-- 
Configure bugmail: http://bugzilla.open-bio.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.