Re: new html-acct-table patch [was Re: html-acct-table patch]

2007-11-01 Thread Andrew Sackville-West
On Thu, Nov 01, 2007 at 09:22:54PM +0100, Christian Stimming wrote: > Am Mittwoch, 31. Oktober 2007 18:34 schrieb Mike Alexander: > > >> This is perfectly fine! Thanks a lot. I've checked some reports here > > >> that turn out identical to before (notably balance-sheet, > > >> account-summary and

Re: new html-acct-table patch [was Re: html-acct-table patch]

2007-11-01 Thread Christian Stimming
Am Mittwoch, 31. Oktober 2007 18:34 schrieb Mike Alexander: > >> This is perfectly fine! Thanks a lot. I've checked some reports here > >> that turn out identical to before (notably balance-sheet, > >> account-summary and income-statement), so hopefully all of the > >> reports are at least as cor

Re: new html-acct-table patch [was Re: html-acct-table patch]

2007-10-31 Thread Andrew Sackville-West
On Wed, Oct 31, 2007 at 05:25:50PM -0400, Mike Alexander wrote: > --On October 31, 2007 1:00:14 PM -0700 Andrew Sackville-West >> per http://www.delorie.com/gnu/docs/guile/guile_252.html, hash-set! >> finds the key in table and assigns the value to that slot. > The description of hash-set! tha

Re: new html-acct-table patch [was Re: html-acct-table patch]

2007-10-31 Thread Mike Alexander
--On October 31, 2007 1:00:14 PM -0700 Andrew Sackville-West <[EMAIL PROTECTED]> wrote: > On Wed, Oct 31, 2007 at 01:34:57PM -0400, Mike Alexander wrote: > >> The main >> difference is to use a hash table instead of a list to save the >> account balances. This cuts another 11% or so off the tim

Re: new html-acct-table patch [was Re: html-acct-table patch]

2007-10-31 Thread Andrew Sackville-West
On Wed, Oct 31, 2007 at 01:34:57PM -0400, Mike Alexander wrote: > The main > difference is to use a hash table instead of a list to save the account > balances. This cuts another 11% or so off the time for my balance sheet > report. > I read up a bit on hash-tables and have some questions/com

Re: new html-acct-table patch [was Re: html-acct-table patch]

2007-10-31 Thread Andrew Sackville-West
On Wed, Oct 31, 2007 at 01:34:57PM -0400, Mike Alexander wrote: ... > > I had started down the same path some time ago but never finished it. Here > is a patch that combines what I did with these changes. The main > difference is to use a hash table instead of a list to save the account > bala

Re: new html-acct-table patch [was Re: html-acct-table patch]

2007-10-31 Thread Mike Alexander
--On October 30, 2007 2:34:14 PM -0700 Andrew Sackville-West <[EMAIL PROTECTED]> wrote: On Tue, Oct 30, 2007 at 10:20:46PM +0100, Christian Stimming wrote: Am Montag, 29. Oktober 2007 22:44 schrieb Andrew Sackville-West: > On Sat, Oct 27, 2007 at 04:27:29PM -0700, Andrew Sackville-West > wrote

Re: new html-acct-table patch [was Re: html-acct-table patch]

2007-10-30 Thread Andrew Sackville-West
On Tue, Oct 30, 2007 at 10:20:46PM +0100, Christian Stimming wrote: > Am Montag, 29. Oktober 2007 22:44 schrieb Andrew Sackville-West: > > On Sat, Oct 27, 2007 at 04:27:29PM -0700, Andrew Sackville-West wrote: > > > On Fri, Oct 26, 2007 at 09:46:03PM +0200, Christian Stimming wrote: > > > > I'd lik

Re: new html-acct-table patch [was Re: html-acct-table patch]

2007-10-30 Thread Christian Stimming
Am Montag, 29. Oktober 2007 22:44 schrieb Andrew Sackville-West: > On Sat, Oct 27, 2007 at 04:27:29PM -0700, Andrew Sackville-West wrote: > > On Fri, Oct 26, 2007 at 09:46:03PM +0200, Christian Stimming wrote: > > > I'd like to apply that patch, however: [...] I'd > > > like to ask to clean up the

new html-acct-table patch [was Re: html-acct-table patch]

2007-10-29 Thread Andrew Sackville-West
On Sat, Oct 27, 2007 at 04:27:29PM -0700, Andrew Sackville-West wrote: > On Fri, Oct 26, 2007 at 09:46:03PM +0200, Christian Stimming wrote: > > I'd like to apply that patch, however: [...] I'd > > like to ask to clean up the comments a bit [...] Is this possible? > > absolutely. I expected as

Re: html-acct-table patch

2007-10-27 Thread Andrew Sackville-West
On Fri, Oct 26, 2007 at 09:46:03PM +0200, Christian Stimming wrote: > Hi Andrew, > > thanks a lot for working on this! I now gave it a try, and your results are > really impressive. The income statement, which currently takes ~45 seconds, > which means it is unusable, is now finished in 10 secon

Re: html-acct-table patch

2007-10-26 Thread Christian Stimming
Hi Andrew, thanks a lot for working on this! I now gave it a try, and your results are really impressive. The income statement, which currently takes ~45 seconds, which means it is unusable, is now finished in 10 seconds, which is at least somehow usable. http://bugzilla.gnome.org/show_bug.cgi

Re: html-acct-table patch

2007-10-18 Thread Andrew Sackville-West
On Tue, Oct 16, 2007 at 02:11:52PM -0700, Andrew Sackville-West wrote: > Hi. > > the attached patch fixes up some of html-acct-table.scm in an effort > (successful!) to speed up the reports that rely on http://bugzilla.gnome.org/show_bug.cgi?id=488001 A signature.asc Description: Digital sign

html-acct-table patch

2007-10-16 Thread Andrew Sackville-West
Hi. the attached patch fixes up some of html-acct-table.scm in an effort (successful!) to speed up the reports that rely on gnc:html-acct-table-add-accounts! (that would be: income-statement, trial-balance, balance-sheet and account-summary). Though this patch doesn't address *all* the issues wi