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

rebase on top of the ethminer master branch #17

Closed
hackmod opened this issue Nov 22, 2018 · 11 comments
Closed

rebase on top of the ethminer master branch #17

hackmod opened this issue Nov 22, 2018 · 11 comments

Comments

@hackmod
Copy link
Contributor

hackmod commented Nov 22, 2018

Experimental

This is my rebase work on top of the latest ethminer master branch.
The master branch of the ethminer a bit faster and the latest ethash already support ProgPoW v0.9.1

Please see
https://github.com/EthersocialNetwork/ethminer-ProgPOW/commits/progpow-master-rebase

Only CUDA tested.

Screen shot

(rebased ProgPoW ethminer + chfast/ethash master branch with a small progpow CPU verifier wrapper fix)
image

image
P106-6G (samsung) ~10.4MH/s (~24.0MH/s for ethash)

@ghost
Copy link

ghost commented Nov 22, 2018

@hackmod @ifdefelse @RadixPi

CI test results:

log:

CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
CUDA_nvrtc_LIBRARY
    linked by target "ethash-cuda" in directory C:/projects/ethminer/libethash-cuda
-- Configuring incomplete, errors occurred!

I think CMake can't find the specific library that is defined here

https://github.com/EthersocialNetwork/ethminer-ProgPOW/blame/master/libethash-cuda/CMakeLists.txt#L42

@hackmod
Copy link
Contributor Author

hackmod commented Nov 29, 2018

@naikmyeong // fixed now
anyway, the openCL version have some period_seed issue

@ghost
Copy link

ghost commented Dec 2, 2018

@hackmod I can't see any test results, is it possible to rebase your fix again?

@hackmod
Copy link
Contributor Author

hackmod commented Dec 9, 2018

rebased again on top of the latest master branch ethereum-mining/ethminer#1766

@hackmod
Copy link
Contributor Author

hackmod commented Dec 10, 2018

updated. rebased again~
support CUDA-8 (compiled. not tested test passed)/CUDA-9/CUDA-10

@hackmod
Copy link
Contributor Author

hackmod commented Dec 13, 2018

binaries are available at https://github.com/hackmod/ethminer/releases/tag/v0.9.2-progpow

  • only CUDA supported
  • you can test it locally with --noevel option like as follow
    ethminer --noeval -v 3 --cuda-devices 0 -U -P http://127.0.0.1:8545

See also progpow testnet:

hackmod pushed a commit to EthersocialNetwork/ethminer-ProgPOW that referenced this issue Dec 14, 2018
@hackmod
Copy link
Contributor Author

hackmod commented Dec 18, 2018

https://github.com/hackmod/ethminer/releases/tag/v0.9.2-opencl-win-1218
OpenCL supported now

  • extract compileKernel() routine and call compileKernel(prog_seed) like as CUDAMiner
  • CUDA-8 support added (GTX 680 confirmed)
  • temporally disable CUDA to strip nvrtc*.(so|dll) dependency

@hackmod
Copy link
Contributor Author

hackmod commented Dec 19, 2018

fixed epoch change issue
https://github.com/hackmod/ethminer/releases/tag/v0.9.2-opencl-win-1219

  • OpenCL: fixed setArg issue after epoch change
  • CUDA: fixed manual rebase mistake. (fix incompatible driver context error after epoch change)

@ifdefelse
Copy link
Owner

ifdefelse commented Jan 15, 2019

Considering https://github.com/AndreaLanfranchi/ethminer is close to ethminer master and runs both ethash and progpow I think we can close this out as fixed. Thanks for all the work.

@solardiz
Copy link
Contributor

solardiz commented Feb 6, 2019

@ifdefelse What's your desired/expected future for this repo vs. https://github.com/AndreaLanfranchi/ethminer? ProgPOW is still being revised in this repo, yet in a tree based off older ethminer. Is the intended use of this repo to serve as reference and for testing? Should those ProgPOW spec and implementation changes be getting ported to https://github.com/AndreaLanfranchi/ethminer and at what points? Thanks!

@ifdefelse
Copy link
Owner

This repro is only intended for algorithm research and development. Andrea's implementation is up-to-date and much more polished for going into production.

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

No branches or pull requests

3 participants