Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • Aug 05 15:15
    mojitoo opened #3732
  • Aug 02 07:34
    s7500 commented #3729
  • Jul 30 08:40
    phyro closed #3730
  • Jul 30 04:08
    cliik commented #3730
  • Jul 28 10:12
    yeastplume synchronize #3695
  • Jul 28 10:12

    yeastplume on pibd_impl

    derive debug for ServerStats (#… Merge branch 'master' into pibd… (compare)

  • Jul 28 10:08

    phyro on master

    derive debug for ServerStats (#… (compare)

  • Jul 28 10:08
    phyro closed #3731
  • Jul 28 09:48
    yeastplume opened #3731
  • Jul 28 09:20
    yeastplume commented #3729
  • Jul 28 09:09
    yeastplume closed #3693
  • Jul 21 11:13
    cekickafa commented #3730
  • Jul 21 11:12
    cekickafa commented #3730
  • Jul 20 08:03
    MCM-Mike commented #3710
  • Jul 19 10:09
    Type1J opened #3730
  • Jul 19 07:41
    davidtavarez commented #3710
  • Jul 14 21:02
    jaw709 commented #3725
  • Jul 14 18:25
    30-r commented #3725
  • Jul 14 18:24
    jaw709 commented #3725
  • Jul 14 18:24
    jaw709 commented #3725
lehnberg
@lehnberg
oh yes it is
Yeastplume
@yeastplume
Right, so if everyone's okay with a proof verification in the next release, (also low risk) I can get that in as well
lehnberg
@lehnberg
I mean...
we're meant to cut a release candidate in three days?
or is that timeline changing?
Antioch Peverell
@antiochp
24 hours of beta.2 or something
Yeastplume
@yeastplume
It's dependent on the sync issues for the most part, the other stuff is minor and can really wait if needed
what's a 'release candidate'?
Antioch Peverell
@antiochp
some eyes on mimblewimble/grin#3164 and mimblewimble/grin#3165 would be good to get the sync issues hopefully resolved
Yeastplume
@yeastplume
I mean, we don't do special RC builds, if it turns out everyone is happy with beta.2 that's the release candidiate
lehnberg
@lehnberg
Release Candidate is like "this is a version we think is good enough for final"
Yeastplume
@yeastplume
I meant in our context
Antioch Peverell
@antiochp
yes I guess the assumption here is beta.2 is the RC (pending anything unexpected)
lehnberg
@lehnberg
And then it's out in the wild until our assumption is proven right
Yeastplume
@yeastplume
Yes, I'm okay with the assumption beta.2 is the RC, until someone demonstrates any reasons why it shouldn't be
Antioch Peverell
@antiochp
beta.1 is the RC currently given the fact we are doing a release for HF2
lehnberg
@lehnberg
oh I see. yeah I mean, generally, if you're doing bug fixes and release, there's a good chance you might get a release candidate if there's no regression
but if you add new functionality and release, you might get a release candidate, or you might get a list of bug fixes that you need to do
Antioch Peverell
@antiochp
yes absolutely - last thing we want to do right now is add to the list of outstanding bugs
Yeastplume
@yeastplume
Okay, so happy with saying no to new features as well
and just including any needed sync fixes
lehnberg
@lehnberg
I feel it's up to you guys as devs to agree on. I'm skeptical myself, but don't feel I've got a horse in the race
my opinions are as per the comment in the thread
https://github.com/mimblewimble/grin/pull/3161#issuecomment-563217208
Antioch Peverell
@antiochp
I'm going to test the TUI changes though locally - they are limited enough in scope I think and give some tangible benefits (no new features)
one fixes a known panic in the tui for example
Yeastplume
@yeastplume
Okay, look let's leave it at that then.
lehnberg
@lehnberg
but I'm not the one that's at risk of being stuck doing last minute troubleshooting & bug fixes over the holidays
Yeastplume
@yeastplume
And on the stuff I'm looking at I'm happy to have at least one definite feature add for 3.0.1
Antioch Peverell
@antiochp
all that said - the TUI is usable right now, so I'm not entirely convinced by my own argument here
Yeastplume
@yeastplume
heh
lehnberg
@lehnberg
I used to be liberal with these things, but a couple of years of releasing turned me into ultra-conservative
Yeastplume
@yeastplume
... okay... Sync fixes + TUI fixes, since they're fixes they make sense to put in
lehnberg
@lehnberg
as I've heard enough "and you won't believe what happened next" war stories
Yeastplume
@yeastplume
and no new features. Yes, I was bitten by 'just a small change' as recently as the 2.1.0 release as well
Antioch Peverell
@antiochp
sync stuff should be fixed if we can wing it though - we've seen enough "locked up sync at step 6/7" for it to be an issue across a large number of nodes
lehnberg
@lehnberg

sync stuff should be fixed if we can wing it though

agree

Antioch Peverell
@antiochp
I just ran 3 successful sync in a row though with a combination of the two PRs up, so I think we have a fix with relatively minor changes involved
Yeastplume
@yeastplume
Okay, so think we're agreed here? Sync fixes necessary, TUI up to @antiochp 's judgement, nothing else?
lehnberg
@lehnberg
and beta2 is the working assumption release candidate?
Antioch Peverell
@antiochp
yes
lehnberg
@lehnberg
ie if all goes well and the roof does not blow up, that's the one
Yeastplume
@yeastplume
:+1:
lehnberg
@lehnberg
🚀
Yeastplume
@yeastplume
And we can branch when it's built
lehnberg
@lehnberg
ah right... for >3.0.0?
Yeastplume
@yeastplume
yes
lehnberg
@lehnberg
sounds good to me
Yeastplume
@yeastplume
already need a place to throw the stuff I'm working on into post 3.0.0, sure others are the same
lehnberg
@lehnberg
do we still believe beta2 could be Thursday?
Antioch Peverell
@antiochp
only node changes for beta.2 or is there wallet changes also?