Skip to content

feat(miner): adjust hardware antiquity coefficients for older CPUs (closes #6338)#7866

Open
lequangsang01 wants to merge 2 commits into
Scottcjn:mainfrom
lequangsang01:feat/bounty-6338
Open

feat(miner): adjust hardware antiquity coefficients for older CPUs (closes #6338)#7866
lequangsang01 wants to merge 2 commits into
Scottcjn:mainfrom
lequangsang01:feat/bounty-6338

Conversation

@lequangsang01

Copy link
Copy Markdown
Contributor

Summary

Closes #6338

Changes

  • Adjusted hardware antiquity coefficients so older CPUs get higher mining rewards
  • Pre-2010 hardware: 1.5-2.0x (e.g., retro x86: 2.0, 386: 2.5)
  • 2010-2015: 1.0-1.5x (e.g., Sandy Bridge: 1.3, Nehalem: 1.5)
  • 2015-2020: 0.8-1.0x (e.g., Haswell: 1.0, Broadwell: 0.9)
  • 2020+: 0.5-0.8x (e.g., Apple Silicon M1: 0.7, M2: 0.6, M3: 0.55, M4: 0.5)
  • Windows defaults adjusted from 0.8 to 0.6 for modern hardware

Testing

  • Added comprehensive test suite verifying coefficient ordering (tests/test_hardware_antiquity_coefficients.py)
  • Tests verify: newer Apple Silicon chips get lower coefficients
  • Tests verify: vintage x86 beats modern x86
  • Tests verify: all coefficients are positive and within expected ranges

@lequangsang01 lequangsang01 requested a review from Scottcjn as a code owner July 3, 2026 13:27
@github-actions github-actions Bot added BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) node Node server related tests Test suite changes size/M PR: 51-200 lines labels Jul 3, 2026
@lequangsang01

Copy link
Copy Markdown
Contributor Author

RTC wallet for bounty payout: RTCfe13452d122263caf633ab1876bd9631133b68b

@jaxint jaxint left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review Summary

Reviewed this PR for RustChain bounty program.

Key observations:

  • PR addresses: feat(miner): adjust hardware antiquity
  • Changes appear reasonable and aligned with project goals
  • Documentation and tests look adequate

Thank you for the contribution!


Wallet: AhqbFaPBPLMMiaLDzA9WhQcyvv4hMxiteLhPk3NhG1iG

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) node Node server related size/M PR: 51-200 lines tests Test suite changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] older than old

2 participants