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

Discrepancy in validators minimum requirements #310

Closed
3 tasks
michalisFr opened this issue Jan 20, 2025 · 0 comments · Fixed by #351
Closed
3 tasks

Discrepancy in validators minimum requirements #310

michalisFr opened this issue Jan 20, 2025 · 0 comments · Fixed by #351
Assignees
Labels
A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) C1 - Medium Medium priority task

Comments

@michalisFr
Copy link

michalisFr commented Jan 20, 2025

In the Reference Hardware section there is a discrepancy between the instances that were used for benchmarking and the minimum specs listed:

  • Instances have 2.6 GHz and 3.5 GHz max boost speed

  • Minimum specs mention 3.4 GHz speed

  • Either the paragraph about the cloud instances needs to be removed (which I don't think serves any purpose) or if the instances' specs are correct (2.6 GHz and 3.5 GHz max boost speed) the minimum specs need to be updated to reflect that.

Also, I'd remove this paragraph:
"While the hardware specs above are best practices and not strict requirements, subpar hardware may lead to performance issues and increase the risk of slashing."

  • I'd give a fix set of minimum requirements that validators need to meet and not leave it open to interpretation.

On that note, I'd suggest:

  • Renaming the section to "Minimum requirements"
@0xLucca 0xLucca self-assigned this Jan 31, 2025
@0xLucca 0xLucca added C1 - Medium Medium priority task A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) labels Jan 31, 2025
@0xLucca 0xLucca linked a pull request Jan 31, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) C1 - Medium Medium priority task
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants