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

Reject ECIP-1093 "RandomX" #359

Closed
wants to merge 1 commit into from
Closed

Reject ECIP-1093 "RandomX" #359

wants to merge 1 commit into from

Conversation

q9f
Copy link
Contributor

@q9f q9f commented Aug 31, 2020

reopen #352

as per reasoning in #333

@gitr0n1n if you want to reiterate this proposal back in draft status, please schedule another call for the sake of process.

@q9f q9f added type: std-core ECIPs of the type "Core" - changing the Classic protocol. status:8 rejected ECIP has been rejected by the community. editor:7 looks-good This contribution is well reviewed and good to be merged. editor:5 needs-review This contribution needs a careful review. and removed editor:7 looks-good This contribution is well reviewed and good to be merged. labels Aug 31, 2020
@gitr0n1n
Copy link
Contributor

Please see the relevant thread to this topic: #352

Copy link
Contributor

@gitr0n1n gitr0n1n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An accepted request from #352:

Please consider leaving this proposal in Draft status as it is my opinion as the author that it never got a fair shake in the ECIP process.

It was rushed through with minimal communication to vested parties. 14 days from inception to reject. Thank you.

There is no harm in leaving it in Draft status for adequate time for discussion. And should ECIP-1049 become contenious, this ECIP will be sitting in the Draft status as an option. If ECIP-1049 is implemented and accept by the network without contention, this proposal will likely be dated and could be rejected at that time. This is the most logical approach as the contra ECIP-1049 sat in draft status for over a year to let the community digest the implications of it.

@q9f
Copy link
Contributor Author

q9f commented Aug 31, 2020

I have nothing else to add.

@gitr0n1n
Copy link
Contributor

gitr0n1n commented Aug 31, 2020

please schedule another call for the sake of process.

I'm happy to schedule this call for right now in Discord where I will read: #352 (comment)

I am sitting in the Discord discussion channel now and will read this message at 8:35 AM UTC. Please attend. Thank you.

@gitr0n1n
Copy link
Contributor

This call has concluded. I am sorry if you could not make the scheduled call.

Here is a brief recap of the ECIP-1093 call just held in the Discussion Calls channel for anyone that missed it:

  • I read this comment out loud: Reject ECIP-1093 "RandomX" #352 (comment)
  • I asked the room if anyone objected to restoring ECIP-1093 back to Draft status.
  • No one objected to this action.
  • I concluded that the call reached rough consensus on the topic with everyone present for the scheduled call.

Thank you to all those who attended.

@q9f
Copy link
Contributor Author

q9f commented Aug 31, 2020

Please don't troll the process. That's the only protection we have.

Moving back to Draft as per comment above.

For further work on RandomX I would kindly ask you to research the same as the SHA3 team and the Keccak256 team:

  • How would a transition look like (1 block, slow mining, hybrid, etc.)?
  • What change in difficulty would be required? How do hashrates compare on different hardware (can it be factorized)?
  • How would ETC hashrate compare to competitors (Monero, etc.)?

@q9f q9f closed this Aug 31, 2020
@q9f q9f deleted the q9-1093 branch August 31, 2020 08:53
@gitr0n1n
Copy link
Contributor

For further work on RandomX I would kindly ask you to research the same as the SHA3 team and the Keccak256 team:

  • How would a transition look like (1 block, slow mining, hybrid, etc.)?
  • What change in difficulty would be required? How do hashrates compare on different hardware (can it be factorized)?
  • How would ETC hashrate compare to competitors (Monero, etc.)?

Please add this ECIP-1093 feedback to the ECIP discussion thread #329 . Thank you for closing this PR.

@developerkevin
Copy link
Member

As this ECIP was rejected on proper grounds, following the ECIP process to a "T", the author, Ronin, must draft a new ECIP while this one stays rejected. Its crucial we don't give one group or individual preferential treatment just because they are incessantly harassing you here and in discord. All he has to do is draft either the same one or make some changes if he wants but it must be a new ECIP to follow the process. Once rejected, an ECIP cannot magically be un-rejected.

@developerkevin
Copy link
Member

Also this process should not be made a mockery of. It's for technical discussion not politics!

@gitr0n1n
Copy link
Contributor

gitr0n1n commented Sep 5, 2020

As this ECIP was rejected on proper grounds, following the ECIP process to a "T", the author, Ronin, must draft a new ECIP while this one stays rejected. Its crucial we don't give one group or individual preferential treatment just because they are incessantly harassing you here and in discord. All he has to do is draft either the same one or make some changes if he wants but it must be a new ECIP to follow the process. Once rejected, an ECIP cannot magically be un-rejected.

I spent a night reviving this ECIP following the instructions from the call moderator. I'm happy to redraft the proposal and submit this proposal again. That is a simply copy and paste effort for me. Actually less effort than having to hold a 60 minute call in the discord that night. So, I have no issue resubmitting this proposal again. Thanks for informing me of the proper ECIP process @developerkevin. It is appreciated.

Here is the PR to resubmit this proposal: #363

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. status:8 rejected ECIP has been rejected by the community. type: std-core ECIPs of the type "Core" - changing the Classic protocol.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants