> Hello all,
> 
> It appears that somebody has broken the average balance script recently;
> kind of annoying because once upon a time I wrote it to serve my own
> purposes, which it no longer does.
 
It was redone a while back by Christopher Browne and myself as part
of collecting a lot of report functionality into report-utilites.
I just noticed the 'min' and 'max' columns are swapped, but other
than that I don't see problems. What does it not do that it used to?


> The most glaring error appears to be the calculation of the current
> balance by adding up the split values even though the calculations are
> not made from day 1 of the account (to do so would be a waste of CPU
> cycles IMHO).
 
Are the values wrong? The report actually runs much faster than it
did before by using better scheme code.


> Similar miscalculations seem to be made in a number of places and I
> would rather not crawl through the code to find all of these mistakes if
> at all possible.

If you can tell me where the mistakes are, I will fix them.


> Is there any way we can pull the previous version(s) out of CVS, at
> least for my own use ?

You can pull out any version you like:

cvs update -D "some date" average-balance.scm


>             It would be REAL nice to be able to use this again...
> 
> If not, can my name please be removed from this mess for which I am not
> responsible ?

Sure.


>     It is a matter of responsibility that contributors to open source
> projects take both credit and blame when seriously modifying code.

If you want someone to blame, you can blame me :)

dave

--
Gnucash Developer's List 
To unsubscribe send empty email to: [EMAIL PROTECTED]


Reply via email to