ethereumproject / ECIPs

The Ethereum Classic Improvement Proposal
55 stars 47 forks source link

Problem: ECIP process not well defined #54

Closed sjmackenzie closed 6 years ago

sjmackenzie commented 7 years ago

Problem: The ECIP process suffers from these issues

Solution: make use of this 2/COSS to manage the ECIP process

sorpaas commented 7 years ago

(@sjmackenzie I would suggest you to break down this PR to some smaller ones (containing C4 and COSS only) if you really want to do something with it.)

I'm involved in C4 right now and we currently apply this to SputnikVM, so I'm probably biased, but I would like to add a few data points to this issue, and encourage some discussions on this issue if you're interested.

For people who are not familiar with this, this basically applies optimistic merging (a.k.a. C4) to ECIPs. The author has an amount of experiences of working in C4 (Fractalide and ZeroMQ). However, we didn't have a good experience in using it right now. From my experience, for this unmodified version of C4 to work, the community needs either:

Our community doesn't have those properties, and we've indeed met several incidents using this for SputnikVM and the projects related:

SputnikVM right now uses 20-C4, an experimental improved version of C4 that aims to solve those problems. But right now we haven't reached anywhere yet. So I would suggest we have some more discussions first here -- what problem does C4 solve for ECIPs? How should C4 be modified to suit ECIPs' need?

realcodywburns commented 7 years ago

I think the consensus oriented system would be a much needed improvement over the current system

splix commented 7 years ago

I also want to avoid formalising the process, in my opinion C4+COSS isn't required there. Also I'd prefer existing naming convention with just number + title (ECIP-1010 Delay Difficulty Bomb Explosion). Because it's simpler, there are no problem with it currently and it's standard convention for most of other cryptocurrencies.