Skip to content

Comments

Standardize hashrate values to Gh/s#1371

Merged
WantClue merged 2 commits intobitaxeorg:masterfrom
mutatrum:normalize-all-hashrates
Nov 24, 2025
Merged

Standardize hashrate values to Gh/s#1371
WantClue merged 2 commits intobitaxeorg:masterfrom
mutatrum:normalize-all-hashrates

Conversation

@mutatrum
Copy link
Collaborator

@mutatrum mutatrum commented Nov 19, 2025

Nice cleanup by pushing the normalization from Gh/s into the hash-suffix-pipe. Basically all hashrates in the code are now in Gh/s. The opposite is hashCounterToGhs from utils.c, so the backend is also completely in Gh/s.

Extracted commit from #1368.

@mutatrum mutatrum changed the title Standardize hashrate values to GH/s Standardize hashrate values to Gh/s Nov 19, 2025
@github-actions
Copy link

github-actions bot commented Nov 19, 2025

Test Results

19 tests  ±0   19 ✅ ±0   0s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit af52952. ± Comparison against base commit 8dc659c.

♻️ This comment has been updated with latest results.

@WantClue WantClue added the cleanup Code cleanup label Nov 19, 2025
@WantClue WantClue added this to the 2.12.0 milestone Nov 19, 2025
@duckaxe duckaxe self-requested a review November 20, 2025 15:22
Copy link
Collaborator

@duckaxe duckaxe left a comment

Choose a reason for hiding this comment

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

Tested on my Ultra.

@WantClue WantClue merged commit d27cd35 into bitaxeorg:master Nov 24, 2025
3 checks passed
@mutatrum mutatrum deleted the normalize-all-hashrates branch November 24, 2025 12:32
mutatrum added a commit to mutatrum/ESP-Miner that referenced this pull request Dec 15, 2025
* Standardize hashrate values to GH/s

* Remove comment
mutatrum added a commit to mutatrum/ESP-Miner that referenced this pull request Dec 15, 2025
* Standardize hashrate values to GH/s

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

Labels

cleanup Code cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants