2021-05-16 GnuCash IRC logs

00:07:41 *** sbluhm has joined #gnucash
00:52:17 *** Aussie_matt has quit IRC
01:21:24 *** frakturfreak has quit IRC
01:34:42 *** Mechtilde has joined #gnucash
01:35:28 *** frakturfreak has joined #gnucash
01:45:53 *** fell has quit IRC
01:47:12 *** fell has joined #gnucash
01:47:12 *** ChanServ sets mode: +o fell
01:47:25 *** Aussie_matt has joined #gnucash
02:01:41 *** sbluhm has quit IRC
02:17:56 *** sbluhm has joined #gnucash
02:17:56 *** ChanServ sets mode: +v sbluhm
03:09:46 *** storyjesse has joined #gnucash
04:05:33 *** User_ has joined #gnucash
04:42:06 *** jervin has quit IRC
04:42:10 *** jervin has joined #gnucash
04:58:13 *** jervin1 has joined #gnucash
04:58:24 *** Mechtilde has quit IRC
04:58:40 *** jervin has quit IRC
04:58:40 *** jervin1 is now known as jervin
05:01:10 *** jervin has quit IRC
05:01:45 *** jervin has joined #gnucash
05:04:05 *** jervin has quit IRC
05:10:04 *** Aussie_matt has quit IRC
05:18:08 *** Mechtilde has joined #gnucash
05:27:19 *** field^Mop has joined #gnucash
06:08:17 *** PowaBanga has quit IRC
06:10:20 *** PowaBanga has joined #gnucash
06:23:56 *** sbluhm has quit IRC
06:32:19 *** sbluhm has joined #gnucash
06:32:19 *** ChanServ sets mode: +v sbluhm
06:42:21 *** storyjesse has quit IRC
07:00:04 *** sbluhm has quit IRC
07:02:34 *** David has quit IRC
07:02:39 *** David has joined #gnucash
07:18:17 *** sbluhm has joined #gnucash
07:18:18 *** ChanServ sets mode: +v sbluhm
07:24:27 *** Aussie_matt has joined #gnucash
07:33:02 *** storyjesse has joined #gnucash
07:43:09 *** storyjesse has quit IRC
08:29:31 *** field^Zzz1 has joined #gnucash
08:31:19 *** field^Mop has quit IRC
08:49:03 *** sbluhm has quit IRC
09:04:46 *** sbluhm has joined #gnucash
09:04:46 *** ChanServ sets mode: +v sbluhm
09:37:15 *** mydogsnameisrudy has joined #gnucash
09:40:35 *** mydogsnameisrudy has quit IRC
09:47:54 <chris> jralls: I think your accountlist issue arises because the scheme accountlist retriever is a lambda, and can be triggered later on.
09:52:36 *** mydogsnameisrudy has joined #gnucash
09:52:46 *** sbluhm has quit IRC
09:53:19 *** mydogsnameisrudy has quit IRC
10:23:59 *** weego has quit IRC
10:28:02 *** Pegasus_RPG has joined #gnucash
10:34:53 *** sbluhm has joined #gnucash
10:34:53 *** ChanServ sets mode: +v sbluhm
10:44:52 *** bertbob has quit IRC
10:48:19 *** storyjesse has joined #gnucash
10:50:14 *** bertbob has joined #gnucash
10:50:14 *** ChanServ sets mode: +v bertbob
10:51:19 *** Aussie_matt has quit IRC
11:06:07 *** bertbob has quit IRC
11:09:42 <chris> also jralls no obj to #998?
11:28:15 *** bertbob has joined #gnucash
11:28:15 *** ChanServ sets mode: +v bertbob
11:52:14 *** mikey has quit IRC
12:08:37 *** sbluhm has quit IRC
12:17:07 <jralls> chris, Sorry, the branch I pushed has a bunch of uncommitted changes, no surprise it won't build. I just pushed that one commit to show the changes I had to make to get gnc:make-account-list-option actually get an account list passed to it. Only those 4 instances were broken.
12:22:08 <jralls> chris, both create-test-data and create-stock-test-data return an alist not a lambda. gnc:report-template-options-generator returns a lambda to gnc:report-template-new-options that IIUC executes it via gnc:backtrace-if-exception.
12:25:18 *** Pegasus_RPG has quit IRC
12:25:58 <jralls> chris, but AFAICT the accounts-alist is never used; what's important is the side effect that create-test-data and create-stock-test-data create child accounts of the root account, used in the account-option generator by calling (gnc-account-get-descendants-sorted (gnc-get-current-root-account)).
12:29:50 <jralls> What I don't understand is how the scheme options code somehow makes the account hierarchy side effect happen when the options generator is run first as in (let * ((options (gnc:make-report-options uuid))(account-alist (create-test-data))...
12:35:15 <jralls> As for PR#998, looks fine to me, though I'll admit some sympathy with gjanssens's argument that it's rewarding one user's loud whining and sets an unfortunate precedent that any UI change requires a preference. That will lead either to a very cluttered preference dialog or to never making any changes that alter existing behavior.
12:48:12 *** sbluhm has joined #gnucash
13:04:41 *** sbluhm has quit IRC
14:10:24 *** Mechtilde has quit IRC
14:10:41 *** Mechtilde has joined #gnucash
14:17:34 *** sbluhm has joined #gnucash
14:17:34 *** ChanServ sets mode: +v sbluhm
14:27:14 *** storyjesse has quit IRC
14:55:26 *** jervin has joined #gnucash
15:12:19 *** sbluhm has quit IRC
15:42:40 *** User_ has quit IRC
15:44:52 *** jervin has quit IRC
15:46:11 *** guak has joined #gnucash
15:48:03 *** jervin has joined #gnucash
15:59:24 *** Mechtilde has quit IRC
16:09:06 *** guak has quit IRC
16:44:01 *** chf has quit IRC
17:42:24 *** chf has joined #gnucash
18:12:26 *** jw4 has quit IRC
18:13:03 *** jw4 has joined #gnucash
18:13:03 *** ChanServ sets mode: +v jw4
18:16:31 *** field^Zzz1 has quit IRC
19:20:18 <chris> jralls: https://pastebin.com/raw/2z3meE9J the default-getter is called twice
19:20:56 <chris> try patching and run any report tester eg test-equity-statement
19:25:27 *** Aussie_matt has joined #gnucash
19:26:03 <jralls> chris, a is empty and b has a list of Account*.
19:27:57 <jralls> That's in current maint. The c++options version of options.scm doesn't have that code.
19:34:10 <jralls> But getter itself is a lambda, so that explains why it's possible to populate the account tree after the option is created: The option doesn't have a value, it has a function that will create a value on demand.
19:35:19 <chris> ^ that was what I meant by 'the retriever is a lambda'
19:37:35 <jralls> OIC.
19:55:54 <jralls> Is there anything gained by that extra complexity?
19:57:59 <chris> probably yes - open a report tab, switch to account tab, create accounts; rerun report - it can pick up new accounts
20:01:45 <jralls> As long as the user hasn't used the account selector to make option-set #t. Hmm.
20:04:09 <chris> ^ this part is probably a wart -- if user uses the account selector, goes to account tab, delete the account, rerun report -- it'll try process NULL account
20:13:23 <jralls> Maybe. Does map include a nil in the result list?
20:13:42 <jralls> Still, better a nullptr than a dangling ptr.
20:13:51 <jralls> Time to go make dinner.
20:14:27 <chris> Not sure - I haven't tested this scenario myself.
20:17:22 <chris> meanwhile my master-chartjs3 branch migrates to chartjs-3.2
20:41:48 *** mikey has joined #gnucash
20:41:48 *** ChanServ sets mode: +v mikey
21:05:18 *** storyjesse has joined #gnucash
21:54:11 *** jervin has quit IRC
23:49:55 *** angel has joined #gnucash