Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move ecip-1047.md to Deferred Status #516

Merged
merged 4 commits into from
Dec 12, 2023
Merged

Move ecip-1047.md to Deferred Status #516

merged 4 commits into from
Dec 12, 2023

Conversation

gitr0n1n
Copy link
Contributor

2023 Update: This was accidentally observed on the network due to a large mining pool setting their default gas to this level. This prevented the development community from the ability to deploy Smart Contracts to Ethereum Classic briefly. Once observed, it was corrected and the mining pool set their gas limit to 8M.

Additionally over three years, seems stale and dated.

ECIP-1000
"ECIPs should be changed from Draft or Last Call status, to Rejected, upon request by any person, if they have not made progress in three years. Such a ECIP may be changed to Draft status if the champion provides revisions that meaningfully address public criticism of the proposal, or to Last Call if it meets the criteria required as described in the previous paragraph."

#14
#253 (comment)

2023 Update: This was accidentally observed on the network due to a large mining pool setting their default gas to this level. This prevented the development community from the ability to deploy Smart Contracts to Ethereum Classic briefly. Once observed, it was corrected and the mining pool set their gas limit to 8M. 

Additionally over three years, seems stale and dated.

ECIP-1000
"ECIPs should be changed from Draft or Last Call status, to Rejected, upon request by any person, if they have not made progress in three years. Such a ECIP may be changed to Draft status if the champion provides revisions that meaningfully address public criticism of the proposal, or to Last Call if it meets the criteria required as described in the previous paragraph."

#14
#253 (comment)
@gitr0n1n gitr0n1n requested a review from a team November 21, 2023 20:33
@meowsbits
Copy link
Member

This is OK with me, although it seems weird for a type=Informational document to even be able to have Rejected|Accepted statuses in the first place. I'd appreciate further feedback on this before going ahead with formal acceptance.

@meowsbits meowsbits added the editor:5 needs-review This contribution needs a careful review. label Dec 5, 2023
@gitr0n1n
Copy link
Contributor Author

gitr0n1n commented Dec 5, 2023

This is OK with me, although it seems weird for a type=Informational document to even be able to have Rejected|Accepted statuses in the first place. I'd appreciate further feedback on this before going ahead with formal acceptance.

When reviewing for this, I felt the same way, but felt confined as this statement appeared to be my only clear option in the ECIP-1000.

ECIPs should be changed from Draft or Last Call status, to Rejected, upon request by any person, if they have not made progress in three years.

While looking at it a bit deeper the ECIP-1000 cites:

An Informational ECIP describes an Ethereum Classic design issue, or provides general guidelines or information to the Ethereum Classic community, but does not propose a new feature. Informational ECIPs do not necessarily represent Ethereum Classic community consensus or a recommendation, so users and implementors are free to ignore Informational ECIPs or follow their advice.

And our available statuses to signal the ECIP is no longer advisable are: Deferred, Rejected, Withdrawn, Replace, Final.

The ECIP editor may also change the status to Deferred when no progress is being made on the ECIP.

Maybe Deferred is more appropriate as this Informational ECIP is not recommended/advised, as it would restrict basic smart contract deployment for the dapp community and, thus, obsolete at this point. Does Deferred seem more appropriate to the other @ethereumclassic/ecip-editors ?

image
https://ecips.ethereumclassic.org/ECIPs/ecip-1000

Change to Deferred status
@gitr0n1n
Copy link
Contributor Author

gitr0n1n commented Dec 6, 2023

Withdrawn status makes the most sense to me. I'm just not confident we, as ECIP editors following the ECIP-1000, have the authority to change to that specific Withdrawn status. I do see a clear statement that we may change to Deferred. In this case, Deferred refers to waiting for the author to confirm Withdrawn or redraft with a more accurate gas recommendation that is relevant to today's chain. Or perhaps this proposal will be Replaced by a more dynamic gas pricing solution to prevent needless bloat.

@IstoraMandiri
Copy link
Contributor

Agree with Deffered for now.

The gas limit in theory makes sense, but as we have seen recently, in practice has negative outcomes particularly with complicating deployments.

Even some existing deployed contracts have embedded assumption of 1M+ transactions, and although I have argued in the past this were a bad assumptions, breaking them would likely to be considered to cause an unnecessary pain (and questioning Code is Law), unless addressing a critical problem with the protocol, which we don't appear to have currently.

Today, I probably wouldn't support capping the gas limit to 1M, not least for the sake of ossification.

@IstoraMandiri IstoraMandiri changed the title Move ecip-1047.md to Rejected Status Move ecip-1047.md to Deferred Status Dec 9, 2023
@bobsummerwill
Copy link
Member

See also https://ecips.ethereumclassic.org/ECIPs/ecip-1084

If we ever did decide to lock down gaslimit voting, I would not now suggest either of the formulas I suggested there, but an absolute fixed amount as Bitcoin has done.

Not less than 8MB though.

This 1MB "recommendation" would have no teeth anyway and makes no sense to keep around, imho.

Maybe contact Anthony and ask if he can withdraw it?

@gitr0n1n
Copy link
Contributor Author

Pinging @pyskell on here as the provided contact. If we don't here anything, let's merge this in a few weeks.

@bobsummerwill
Copy link
Member

I messaged @pyskell on Telegram and he said that he cannot even remember the issue.

"Yeah feel free to do what you want in that repo and to take me off of it"

I asked whether I could relay his consent and he said "sure".

So I would suggest that the Editors change the status to Withdrawn.

@gitr0n1n gitr0n1n merged commit ba2b67f into master Dec 12, 2023
4 checks passed
@gitr0n1n gitr0n1n deleted the gitr0n1n-patch-2 branch December 12, 2023 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editor:5 needs-review This contribution needs a careful review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants