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

try to explain nonce space duration from paramters #228

Merged
merged 4 commits into from
Jun 19, 2024

Conversation

Georges760
Copy link
Collaborator

try to calculate the asic_job_frequency_ms dynamicaly, to be discussed

@Georges760 Georges760 marked this pull request as draft June 17, 2024 17:49
Copy link
Owner

@skot skot left a comment

Choose a reason for hiding this comment

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

This is looking good to me. Tested on 204, 401 and 402. Way less invalid work and rejected shares

@Georges760 Georges760 marked this pull request as ready for review June 18, 2024 17:47
@Georges760
Copy link
Collaborator Author

I will try it on 302 @WantClue sent me, then merge it if OK

@skot
Copy link
Owner

skot commented Jun 18, 2024

My bitaxeMax (v2.2) isn't working. It never starts hashing. It's been a long time since I tested this, so it might not be this branch causing the problem..

@skot
Copy link
Owner

skot commented Jun 19, 2024

My bitaxeMax (v2.2) isn't working. It never starts hashing. It's been a long time since I tested this, so it might not be this branch causing the problem..

fixed this with e7a29f8.
Tested on 102 (v2.2), 204, 401, and 402

@Georges760
Copy link
Collaborator Author

Georges760 commented Jun 19, 2024

regression came from #192, more specifically 8508862, not to point at anybody, but usually it is wise to look around for any other regression around....

@Georges760
Copy link
Collaborator Author

we can merge it now as it fix BitaxeMax, the Hex will be tested on top of it

@Georges760 Georges760 merged commit 9dc8c8c into master Jun 19, 2024
2 checks passed
@Georges760 Georges760 deleted the nonce_space_duration branch June 19, 2024 13:29
tommywatson pushed a commit to tommywatson/ESP-Miner that referenced this pull request Jun 20, 2024
* try to explain nonce space duration from paramters

* Fix Nonce Space duration for BM1397 (no version-rolling)

* fixed issue with version mask on 1397. added easy serial debugging on 1397

* cleanup jobID debugs

---------

Co-authored-by: Skot <[email protected]>
skot added a commit that referenced this pull request Jun 20, 2024
* Fixed fan speed web update #141

These changes fix fan rpm/percent requested and update both on the web

* fix readme

* refactor self_test to be modular for new hardware

* Supra 402 (#221)

* port TCH Supra 402 branch

* refactor TMP1075 (unused?) driver using i2c_master module

* pulled in @BitMaker-hub stratum_task.c DNS changes from PR #185

* removing serial debug

---------

Co-authored-by: Skot <[email protected]>

* adjust share accpeted/rejected functions to take higher level GLOBAL_STATE to fix share accounting.

* Code clean resulting from looking into #218 (#220)

* Code clean resulting from looking into #218

* Fixed asic count

Set canary value for invalid device's asic_count

---------

Co-authored-by: tommy <[email protected]>

* fix another pointer error

* Changes efficiency metric display in AxeOS (#231)

Fixes #230

* try to explain nonce space duration from paramters (#228)

* try to explain nonce space duration from paramters

* Fix Nonce Space duration for BM1397 (no version-rolling)

* fixed issue with version mask on 1397. added easy serial debugging on 1397

* cleanup jobID debugs

---------

Co-authored-by: Skot <[email protected]>

* Update bm1397.c to increase the max frequency to 650Mhz (#209)

* Update bm1397.c to increase the max frequency to 650Mhz

The original version was setting everything above 500Mhz to 500Mhz, the update increases the limit to 650Mhz.
No changes to the web interface - drop-down still shows up to 575Mhz

* Update edit.component.ts to include higher freqeuncy for BM1397

* Updated BM1397 frequencies to above 500Mhz

* Update bm1397.c

* Update bm1397.c

* UN-Update readme.md

* Update bm1397.c

* Update bm1397.c

* Update bm1397.c

* fix: add recovery page (#232)

Adds a recovery web interface to enable users
to recover from a failed www.bin update.
Partial fix for Issue #213.

* refactor: unify merge_bin scripts (#189)

Combines the functionality of merge_bin_update.sh
and merge_bin_with_config.sh into merge_bin.sh.
Also adds more verbose usage printing.

* fix: check www.bin size before updating (#216)

Adds a basic sanity check for www.bin uploading.
Returns 400 if upload is attempted on a file larger
than the available partition space.

---------

Co-authored-by: tommy <[email protected]>
Co-authored-by: Georges Palauqui <[email protected]>
Co-authored-by: Skot <[email protected]>
Co-authored-by: Nathan Day <[email protected]>
Co-authored-by: yanir99 <[email protected]>
Co-authored-by: tdb3 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants