2023-04-28 GnuCash IRC logs

01:17:47 *** mosiu has joined #gnucash
01:25:52 *** fell has quit IRC
01:27:12 *** fell has joined #gnucash
01:27:12 *** ChanServ sets mode: +o fell
02:07:49 *** gandalf has joined #gnucash
02:28:01 *** NoobAlice has quit IRC
02:34:26 *** gjanssens has joined #gnucash
02:34:26 *** ChanServ sets mode: +o gjanssens
03:30:29 *** gjanssens has quit IRC
03:51:14 *** Simon has quit IRC
03:55:13 *** Simon has joined #gnucash
03:55:13 *** ChanServ sets mode: +v Simon
04:13:00 *** gandalf has joined #gnucash
04:47:52 *** emirhg has joined #gnucash
05:23:51 *** gandalf1 has joined #gnucash
05:24:19 *** gandalf has quit IRC
05:24:19 *** gandalf1 is now known as gandalf
05:32:55 *** emirhg has quit IRC
06:31:17 *** gandalf1 has joined #gnucash
06:31:57 *** gandalf has quit IRC
06:31:57 *** gandalf1 is now known as gandalf
06:46:53 *** chris has joined #gnucash
06:46:54 *** ChanServ sets mode: +v chris
06:46:57 *** gncbot sets mode: +o chris
06:51:49 <chris> @tell gjanssens and jralls any thoughts on #1617 #1618 ? the #1618 is straightforward (IMV) and #1617 seems safe. after 5.1.
06:51:49 <gncbot> chris: The operation succeeded.
07:40:49 *** gandalf1 has joined #gnucash
07:41:16 *** gandalf has quit IRC
07:41:16 *** gandalf1 is now known as gandalf
07:54:12 *** mydogsnameisrudy has joined #gnucash
07:56:02 *** mydogsnameisrudy has quit IRC
08:40:09 *** gjanssens has joined #gnucash
08:40:09 *** ChanServ sets mode: +o gjanssens
09:00:41 *** gandalf1 has joined #gnucash
09:01:10 *** gandalf has quit IRC
09:01:10 *** gandalf1 is now known as gandalf
10:00:50 *** gandalf1 has joined #gnucash
10:01:17 *** gandalf has quit IRC
10:01:17 *** gandalf1 is now known as gandalf
10:20:39 *** gjanssens has quit IRC
10:40:20 *** gjanssens has joined #gnucash
10:40:20 *** ChanServ sets mode: +o gjanssens
11:14:59 *** gandalf1 has joined #gnucash
11:15:26 *** gandalf has quit IRC
11:15:26 *** gandalf1 is now known as gandalf
11:43:15 *** NoobAlice has joined #gnucash
12:13:34 *** gandalf1 has joined #gnucash
12:14:01 *** gandalf has quit IRC
12:14:01 *** gandalf1 is now known as gandalf
12:32:51 *** jervin has quit IRC
12:33:35 *** jervin has joined #gnucash
12:54:12 *** palerider has joined #gnucash
12:54:16 *** palerider has left #gnucash
13:01:28 *** jervin has quit IRC
13:02:41 *** jervin has joined #gnucash
13:11:38 *** gandalf1 has joined #gnucash
13:12:05 *** gandalf has quit IRC
13:12:05 *** gandalf1 is now known as gandalf
13:39:07 *** raeburn has quit IRC
13:48:49 *** warlord2 has quit IRC
13:50:55 <jralls> chris, 1617 is still marked as a draft. 1618 is a big improvement in readability, I think you can merge it when ready.
13:51:31 <jralls> @gjanssens, chris asked you to look at those too with an at-tell just before you logged in.
13:51:31 <gncbot> jralls: Error: "gjanssens," is not a valid command.
13:51:39 <jralls> gjanssens, chris asked you to look at those too with an at-tell just before you logged in.
13:52:37 <jralls> chris, Didn't notice before, you've got 1618 marked as draft too.
13:57:52 *** jervin has quit IRC
13:57:58 *** warlord has joined #gnucash
13:57:58 *** gncbot sets mode: +o warlord
14:00:00 *** ns12 has quit IRC
14:00:10 <gjanssens> I'll see if I still find time for it
14:00:10 <gncbot> gjanssens: Sent 7 hours and 8 minutes ago: <chris> and jralls any thoughts on #1617 #1618 ? the #1618 is straightforward (IMV) and #1617 seems safe. after 5.1.
14:00:25 <warlord> .
14:00:34 *** ns12 has joined #gnucash
14:00:58 <jralls> gjanssens, thanks for finding time to work out the CSV import issues.
14:03:47 *** jervin has joined #gnucash
14:04:11 *** gandalf1 has joined #gnucash
14:04:39 *** gandalf has quit IRC
14:04:39 *** gandalf1 is now known as gandalf
14:23:01 *** Kaligula has joined #gnucash
14:23:07 *** ChanServ sets mode: +v Kaligula
14:53:07 *** jralls_afk has joined #gnucash
14:54:30 *** jralls_afk has quit IRC
14:54:58 *** jralls_afk has joined #gnucash
14:55:11 *** jralls has quit IRC
14:59:04 *** warlord has quit IRC
14:59:43 *** jralls has joined #gnucash
14:59:43 *** ChanServ sets mode: +o jralls
14:59:43 *** jralls_afk has quit IRC
15:00:33 *** jralls_afk has joined #gnucash
15:00:34 *** jralls has quit IRC
15:01:43 *** jervin has quit IRC
15:02:26 *** jervin has joined #gnucash
15:03:26 *** mosiu_ has joined #gnucash
15:03:39 *** mosiu has quit IRC
15:03:40 *** mosiu_ is now known as mosiu
15:07:08 *** jralls has joined #gnucash
15:07:08 *** ChanServ sets mode: +o jralls
15:07:08 *** jralls_afk has quit IRC
15:09:49 *** jralls_afk has joined #gnucash
15:09:49 *** jralls has quit IRC
15:11:22 *** jralls has joined #gnucash
15:11:22 *** ChanServ sets mode: +o jralls
15:13:51 *** jralls_afk has quit IRC
15:14:23 *** raeburn has joined #gnucash
15:14:23 *** ChanServ sets mode: +v raeburn
15:15:10 *** gandalf1 has joined #gnucash
15:15:36 *** gandalf has quit IRC
15:15:36 *** gandalf1 is now known as gandalf
15:15:55 *** jralls has quit IRC
15:16:22 *** jralls has joined #gnucash
15:16:22 *** ChanServ sets mode: +o jralls
15:41:04 *** miklcct has quit IRC
15:46:28 *** miklcct has joined #gnucash
15:46:28 *** ChanServ sets mode: +v miklcct
15:52:57 *** gandalf has quit IRC
16:03:20 *** jralls_afk has joined #gnucash
16:05:25 *** jralls has quit IRC
16:25:30 *** jralls_afk is now known as jralls
16:26:24 *** ChanServ sets mode: +o jralls
16:28:37 *** jervin has quit IRC
16:29:20 *** jervin has joined #gnucash
16:32:32 <gjanssens> jralls: you're welcome
16:46:41 <gjanssens> chris: short first feedback - I agree with jralls 1618 improves readability a lot. So looks like a good improvement if you deem it ready.
16:47:39 <gjanssens> 1617 I have only looked at the description so far and that as well looks like it will be a nice improvement. I can't just right now whether the code is good quality, but tbh I think you're most experienced to judge that.
16:53:04 *** gjanssens has quit IRC
17:11:32 *** Kaligula has quit IRC
17:13:50 *** Kaligula has joined #gnucash
17:13:57 *** ChanServ sets mode: +v Kaligula
17:14:44 *** mp has joined #gnucash
17:19:46 *** mp has quit IRC
17:28:46 *** mosiu has quit IRC
17:43:55 *** jervin1 has joined #gnucash
17:44:22 *** jervin has quit IRC
17:44:22 *** jervin1 is now known as jervin
17:47:20 *** raeburn has quit IRC
18:02:35 *** jervin has quit IRC
19:17:12 *** raeburn has joined #gnucash
19:17:12 *** ChanServ sets mode: +v raeburn
19:19:47 *** Kaligula has quit IRC
19:36:17 *** jralls_afk has joined #gnucash
19:36:17 *** jralls has quit IRC
19:36:38 *** raeburn has quit IRC
19:41:33 *** jralls has joined #gnucash
19:41:33 *** ChanServ sets mode: +o jralls
19:41:33 *** jralls_afk has quit IRC
19:45:54 *** jralls_afk has joined #gnucash
19:46:08 *** jralls has quit IRC
19:46:35 *** jralls has joined #gnucash
19:46:35 *** ChanServ sets mode: +o jralls
20:09:48 <chris> jralls #1618 is increased readability yes, but also introduces a change in the calculated-cells calc-fn (i.e. how do determine monetary amount from a given split)
20:10:05 <chris> previously it accepts split only, now it accepts split and transasction-row? which is a bool
20:10:26 <chris> the latter part is required for #1617
20:12:11 <chris> so I'll do the trep upgrade now and we can work on the 1617 at leisure
20:21:33 <chris> jralls: I'd have used FeatureVec because std::set has a specific meaning in cpp :)
20:25:37 <chris> #1618 has a specific concern of introducing 3 more options. it's otherwise well written though.
20:28:01 <chris> the 3 options introduce 3 further columns: prime-running-bal, sec-running-bal, total-running-bal
23:56:16 *** mosiu has joined #gnucash