Thanks for clarifying that equity was probably a coding error. I also managed to find the original commit message 190665ac which confirms. I'll try to redo, and ensure that the summary lines are as follows... I think they should be insensitive to the sign-reversal preference.
"2) GncBudgetView changes: a) If an account has children but does not have a specific budget value for a period, the sum of the children's budget values will be displayed in gray b) Totals column on the right provides total for a budget row (account) c) At the bottom, totals lines with total income, total expenses, total transfers (assets+liabilities) and grand total (income - expenses - transfers)" On Fri, 3 Jan 2020 at 15:35, Phil Longstaff <phil.longst...@gmail.com> wrote: > I can't think of any reason that equity needs to be included. > > On Fri, Jan 3, 2020 at 9:54 AM Christopher Lam <christopher....@gmail.com> > wrote: > > > Thank you Phil. Github isn't logged anywhere apart from web interface, > > hence not the best for discussion. > > > > The only hiccup I really want to elucidate is whether equity being part > of > > Transfer line is needed at all. So please let me know whether it should > be > > part of the equation at all. > > > > My suggestions for summary lines as follows would encourage users to > think > > 'incoming funds --> outgoing expenses, and invest into savings and pay > off > > loans" > > > > "Inflow from Income" = normally positive > > "Outflow to Expenses" = normally positive > > "Outflow to Asset&Liability" = normally positive > > "Unbudgeted balance" = aiming towards zero > > > > On Fri, 3 Jan 2020 at 11:32, Phil Longstaff <phil.longst...@gmail.com> > > wrote: > > > >> I will continue this discussion here because I can no longer comment on > >> the > >> change on github. > >> > >> No, I understand you aren't trying to be difficult. > >> > >> I think when I made the original change, my books were set with a > certain > >> sign-reversal setting and I didn't test with other settings. My problem > >> now > >> is that I have very little time to spend to fix this. It has been a few > >> years since I have done anything on the gnucash code base and no longer > >> have a working development environment. > >> > >> Phil > >> > >> On Thu, Jan 2, 2020 at 6:04 AM Christopher Lam < > notificati...@github.com> > >> wrote: > >> > >> > P.S. I'm not intending to be difficult here -- the previous strategy > had > >> > assumed that all budget values asset/liability/income/expense/equity > had > >> > assumed sign-reversal=credit-balances, and was adding/subtracting them > >> as > >> > such. After bug fix, if this commit, the summary totals would have > been > >> > extremely broken e.g. asset=$300, liability=$1000; the transfer column > >> > would have shown $1300 or $700 according to sign-reversal preference, > >> and > >> > the total column would be much more wild. > >> > > >> > — > >> > You are receiving this because you were mentioned. > >> > Reply to this email directly, view it on GitHub > >> > < > >> > https://github.com/Gnucash/gnucash/commit/775200ef348f8579a3e0258988ae25a8026dbd6d?email_source=notifications&email_token=AE6Z25I2JTMG4J3KIZ26GT3Q3XC4FA5CNFSM4KB3HZ4KYY3PNVWWK3TUL52HS4DFVVBW63LNNF2EG33NNVSW45FKMNXW23LFNZ2F62LEZYBC6A6C#commitcomment-36635586 > >> >, > >> > or unsubscribe > >> > < > >> > https://github.com/notifications/unsubscribe-auth/AE6Z25OEXIQVNT5ACQEDTBTQ3XC4FANCNFSM4KB3HZ4A > >> > > >> > . > >> > > >> _______________________________________________ > >> gnucash-devel mailing list > >> gnucash-devel@gnucash.org > >> https://lists.gnucash.org/mailman/listinfo/gnucash-devel > >> > > > _______________________________________________ > gnucash-devel mailing list > gnucash-devel@gnucash.org > https://lists.gnucash.org/mailman/listinfo/gnucash-devel > _______________________________________________ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel