Development meeting held @ 3PM UTC in grincoin#general channel on Keybase. Meeting lasted ~ 40 min.
Notes are truncated, and conversations sorted based on topic and not always chronological. Quotes are edited for brevity and clarity, and not always exact.
Community attendance:
- antiochp
- johndavies24
- joltz
- kurt2
- lehnberg
- quentinlesceller
- yeastplume
(apologies if I missed someone - submit a PR or contact @lehnberg to add)
The proposed agenda was accepted, with the point "jaspervdm update" added, and a "4.0.0 beta" point added to Other Questions.
None.
- lehnberg: I had a chance to chat with Jasper just before this meeting. He's alright, wanted to pass on to the meeting that he's taking a leave of absence for personal reasons.
- 👍: antiochp, quentinlesceller, johndavies24
- lehnberg: There's no clear timeframe for how long he will be away, but he's keen on coming back at some point. He also wanted to make clear that he intends to honour his commitments as per his currently active funding period upon his return. (Somehow, exactly how tbd). Any questions about that?
- johndavies24: Wish him the best!
- 👍: kurt2
- antiochp: Looking forward to having him back with us.
- 👍: quentinlesceller
- lehnberg: Seconded to both.
- 👍: joltz, yeastplume
- johndavies24: Wish him the best!
- lehnberg: So this means we should probably spend some extra time thinking about what this means for 4.0.0.
4. GUI wallets
- yeastplume: I am sorry, I have had absolutely no time or headspace to think about it since the previous meeting.
- joltz: 🔔 for me, haven't done much there since last meeting, mainly focused on slatepack stuff. Hope to return to this after release.
- lehnberg: Okay, so let's table this, create a grin-pm issue about it, and track it using the regular issue tracker?.
- 👍: quentinlesceller
- joltz: Sounds good- this is definitely important and we need to get around to it at some point.
- yeastplume: Sounds good, I really do want to get to this but the hf needs are more pressing.
- 👍: joltz, quentinlesceller, lehnberg
- lehnberg: In order of appearance: https://github.com/mimblewimble/grin-rfcs/pulls
5.1 Slatepack
- joltz: I should have slatepack ready for FCP by end of the week - just need to add a few last details. Hoping to have a brief focused security review before final release - maybe a 10-25 hour initial assessment to catch most obvious issues. Not sure if it makes sense to try to do a request for funding or just call in a favor considering the limited timeline.
- lehnberg: A brief slatepack security review would be awesome.
- 🚀: antiochp
- lehnberg: A brief slatepack security review would be awesome.
- lehnberg: Ideally, the RFC would be agnostic to the outcome of that, but maybe that's wishful thinking? I.e. Impact of security review would be related to implementation?
- joltz: Yeah this is more in the impl side. But may have impacts in RFC.
- yeastplume: Well, the whole wallet could do with a security audit. And I'm sure the node does as well at this stage.Perhaps there is a bigger task here to plan for 5.0.0?
- joltz: I agree whole wallet needs audit badly but I do not think we would have time before we would want slatepack reviewed.
- joltz: (though made RFC with simple primitives as possible so shouldn't be much changed).
- lehnberg: Yeah. It's okay to make minor changes to an accepted RFC that does not "alter the spirit" of it. So if something comes up, perhaps it's okay to update RFC accordingly? Assuming the whole thing is not thrown out for a different scheme.
- lehnberg: Okay so I guess... RFC can move forward independently of the (separate) question of a security review (for what and when).
- 👍: joltz
- joltz: Would be really nice to do another large audit before final hf.
- lehnberg: @quentinlesceller as shepherd are you onboard with getting slatepack RFC to FCP this week?
- quentinlesceller: Yes absolutely.
- 🚀: lehnberg
- quentinlesceller: Yes absolutely.
- lehnberg: As discussed, we're okay to look at this after hf. Obviously any progress / feedback can occur now as well.
- 👍: joltz, yeastplume
5.3 Parallel IBD
- lehnberg: So given Jasper's absence as per above, I reckon this will be inactive for the time being.
- antiochp: PIBD on hold but nothing blocking hf3.
- 👍: lehnberg
5.4 N(e)rd kernels
- lehnberg: This has had a very generous time in fcp. Been a few tweaks a while back from @antiochp. There's been no more feedback so I reckon I'll go ahead and merge later today?
- 👍: antiochp, joltz, quentinlesceller
5.5 qa team
- joltz: My understanding was plan was to close that RFC due to interest/resources. It is good to have in theory but in practice may be spread too thin currently to support.
- lehnberg: It's easy to re-submit a QA RFC, even copy/pasting what we had there. If anyone ever feels like taking the lead. I don't think anyone is explicitly against it, more a matter of someone rising up and working through the open questions and turning it into something actionable.
- 👍: joltz, quentinlesceller
- lehnberg: In jasper's absence, I'm happy to step up as shepherd there, and get this bad boy over the finish line.
- yeastplume: That's good to merge I think, as soon as FCP expires.
- lehnberg: Let me be the judge of that, haha. rolls up sleeves
- yeastplume: Feel free to rewrite from the ground up. Quoting your favorite economists and ethical philosophers.
-
lehnberg: Re: IBD issue, am I correct in thinking that's not an issue whatsoever?.
- antiochp: IBD work can continue after hf3 - timing was just for convenience (rolling out new api endpoints etc.).
-
quentinlesceller: Before going deep into 5.0.0 I'd like to propose a grin-wallet v3.1.2 which fix the tor send on the v3.1.1. PR is ready: mimblewimble/grin-wallet#425 What I need is grin and grin-wallet v3.1.1 on crates.io. I'll happily do it if someone give me access.
- yeastplume: @quentinlesceller I can probably give you access to all the crates, pm me user.
- 👍: quentinlesceller, lehnberg
- yeastplume: @quentinlesceller I can probably give you access to all the crates, pm me user.
-
yeastplume: For the beta, there's one pr to merge on the grin side, one on the wallet side, then I think we can do a beta1 and write a post with skeletal release notes outlining things to test. We can get all this done over the next 24 hours. A lot of stuff changed in a very short period on the wallet command line to accommodAte slatepack, and the earlier all the permutations gets in use and start being tested, the better. There are one or two more additions I want to make over the next couple of weeks, but nothing major. But probably quite a few bugs to iron out.
- 👍: lehnberg
- quentinlesceller: I have some bandwidth to test and help @antiochp @yeastplume.
- lehnberg: Sounds good. Where are we with tromp's hardfork3 fork? Https://github.com/mimblewimble/grin/pull/3335.
- lehnberg: Is that the one we're waiting for on the node side?
- yeastplume: Yes.
- lehnberg: What about antioch's mimblewimble/grin#3302. Is that what he meant does not hold up beta?
- quentinlesceller: This one still wip?
- lehnberg: Ah that's the second pr, right?
- antiochp: Yes that one.
- lehnberg: Right. Okay, so yeastplume will look to cut betas by thursday? Both for node and wallet?
- yeastplume: Yep, earlier if possible.
- lehnberg: Fantastic 👍.
- quentinlesceller: 👍 thank you.
Meeting adjourned.