Skip to content

Auto-Tuning Capabilities #1152

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 104 commits into
base: master
Choose a base branch
from
Open

Auto-Tuning Capabilities #1152

wants to merge 104 commits into from

Conversation

@KillerInk
Copy link
Contributor

@KillerInk KillerInk commented Jul 17, 2025

What This Does

This PR introduces a new set of features that significantly enhance the auto-tuning capabilities of the ESP-Miner. The auto-tune functionality is crucial for optimizing mining performance by adjusting key parameters such as power limit, fan speed, voltage, and frequency based on real-time conditions.

Auto-Tune Enhancements:

  • Dynamic Power Management: Users can now adjust the power limit to ensure optimal energy usage while maintaining high hashrates.
  • Temperature Control: Fine-tuning fan limits allows for better thermal management, preventing overheating and potential hardware damage.
  • Voltage and Frequency Adjustments: The ability to modify step voltage and frequency settings helps in achieving the best possible performance from ASICs under varying conditions.

Benefits:

  • Optimized Performance: By dynamically adjusting parameters, users can maximize mining efficiency and profitability.
  • Improved Hardware Longevity: Better temperature control and power management contribute to the longevity of the mining hardware.
  • Remote Configuration: The API-driven approach allows for remote adjustments, making it easier to manage multiple miners or integrate with external monitoring systems.

By enhancing the auto-tuning features, this PR aims to provide users with more control over their mining operations, leading to improved performance and potentially higher returns.

@github-actions
Copy link

github-actions bot commented Jul 17, 2025 •
edited
Loading

Test Results

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

Results for commit 05ec3d0. ± Comparison against base commit 9976a12.

♻️ This comment has been updated with latest results.

@KillerInk
Copy link
Contributor Author

KillerInk commented Jul 18, 2025

this is related to #1099

@KillerInk KillerInk marked this pull request as draft July 18, 2025 03:47
@KillerInk
Copy link
Contributor Author

KillerInk commented Jul 18, 2025

have to extend the chart, that it shows the applied voltage, frequency and fanspeed to have somekind of feedback for the user what is is happening.

@duckaxe
Copy link
Collaborator

duckaxe commented Jul 20, 2025

Still draft?

@KillerInk
Copy link
Contributor Author

KillerInk commented Jul 20, 2025

Yes hope I have tomorrow time to extend the chart and add the missing stuff on esp side to statistic and livedata.
Think that my current chart is a little overdone and contains to many datasets.
Or I keep it as it is and add it as extra menu item where the dashboard, pool, update items are.
But then I still have to cleanup unused datasets.
I'm open to every suggestion

@KillerInk KillerInk marked this pull request as ready for review July 21, 2025 10:06
@KillerInk
Copy link
Contributor Author

KillerInk commented Jul 21, 2025

Unbenannt-1

duckaxe
duckaxe requested changes Jul 23, 2025
View reviewed changes
@duckaxe
Copy link
Collaborator

duckaxe commented Jul 23, 2025

Maybe it's a good idea to wait for #955 before any changes on the home chart?

@KillerInk
Copy link
Contributor Author

KillerInk commented Jul 24, 2025

Maybe it is, no hurry. will keep this up to date with master

@KillerInk
Copy link
Contributor Author

KillerInk commented Jul 24, 2025

well the chart part is more or less obsolete with #955
so i can drop it an keep only the settings part

duckaxe
duckaxe reviewed Nov 3, 2025
View reviewed changes
KillerInk and others added 22 commits November 4, 2025 14:15
@KillerInk
code align
8353fab
@KillerInk
Merge remote-tracking branch 'killerink/master' into master_autoclock
1bcb02e 
# Conflicts:
#	main/http_server/axe-os/src/app/components/home/home.component.ts
#	main/tasks/statistics_task.h
@KillerInk
Merge branch 'master' into master_autoclock
e8512e9
@KillerInk
Merge branch 'master' into master_autoclock
a7e69b0
@KillerInk
Merge remote-tracking branch 'killerink/master' into master_autoclock
9bcb9e4 
# Conflicts:
#	main/http_server/http_server.c
#	main/nvs_config.c
#	main/tasks/power_management_task.c
@KillerInk
Merge remote-tracking branch 'killerink/master' into master_autoclock
ce00cd7
@KillerInk
use power limit from info.maxpower, use max voltage from asic.default…
42fcbcd 
…Voltage *0.25
@KillerInk
change voltage precisions to 2
dbaa819
@KillerInk
floating frequency
5f65ca8
@KillerInk
workaround hashrate drop 0 on frequency change
36be93a
@KillerInk
return on frequency change and ignore register for this round
b978fba
@KillerInk
Revert "return on frequency change and ignore register for this round"
36047b0 
This reverts commit b978fba.
@KillerInk
Revert "workaround hashrate drop 0 on frequency change"
a4cab83 
This reverts commit 36be93a.
@KillerInk
Merge remote-tracking branch 'killerink/master' into master_autoclock
767fd58
@KillerInk
drop vr min max limits
a40e542
@KillerInk
use device specific max values for frequency and voltage
830dae5
@KillerInk
Merge branch 'master' into master_autoclock
a55d0eb
@KillerInk
Merge branch 'master' into master_autoclock
b2f5bb3
@KillerInk
increase voltage precision to 3
97ee116
@KillerInk
poll every 1000ms
0a0c195
@KillerInk
tune against error
3c27720
@KillerInk
Merge remote-tracking branch 'killerink/master_autoclock' into master…
638c63e 
…_autoclock
Evanito
Evanito approved these changes Nov 19, 2025
View reviewed changes
Copy link

@Evanito Evanito left a comment

Choose a reason for hiding this comment

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

Tested and working consistently on Gamma 602

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

Reviewers

@duckaxe duckaxe duckaxe requested changes

@mutatrum mutatrum mutatrum requested changes

+2 more reviewers

@0xf0xx0 0xf0xx0 0xf0xx0 requested changes

@Evanito Evanito Evanito approved these changes

Reviewers whose approvals may not affect merge requirements

Requested changes must be addressed to merge this pull request.

Assignees

@mutatrum mutatrum

@WantClue WantClue

Labels

enhancement New feature or request help wanted Extra attention is needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@KillerInk @duckaxe @chillingbee @Evanito @0xf0xx0 @kakulukia @mutatrum @WantClue