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.

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

feat: cuda miner support #167

Closed
wants to merge 26 commits into from
Closed

feat: cuda miner support #167

wants to merge 26 commits into from

Conversation

hunjixin
Copy link
Contributor

Review Type Requested (choose one):

  • Glance - superficial check (from domain experts)
  • Logic - thorough check (from everybody doing review)

Summary

Provide a one line summary and link to any relevant references

Task/Issue reference

Closes: add_link_here

Details (optional)

Add any additional details that might help Code Reviewers digest this PR

How to test this code? (optional)

Anything else? (optional)

@hunjixin hunjixin marked this pull request as draft June 18, 2024 15:19
@hunjixin
Copy link
Contributor Author

hunjixin commented Jun 18, 2024 •
edited
Loading

issue

  1. need cuda to build binary, so the ci/cd need change
  2. grid and block argument should change according model nvidia cpu, affecting the GPU Parallelism number, also for better hash performance
  3. difficulty maybe need to change adjusting to che cpu speed
  4. very low efficiency

@hunjixin
Copy link
Contributor Author 

lilypad resource-provider --num-worker 40 --web3-private-key 0x8b3a350cf5c34c9194ca85829a2df0ec3153be0318b5e2d3348e872092edffba --service-solver 0x3C44CdDdB6a900fa2b585dd299e03d12FA4293BC -n dev --enable-pow]

@hunjixin
Copy link
Contributor Author

build reference https://github.com/gorgonia/cu

@hunjixin hunjixin force-pushed the feat/cuda_miner_support branch from cdbe6e7 to 46bd465 Compare June 19, 2024 17:07
@hunjixin hunjixin force-pushed the feat/cuda_miner_support branch from 52ce0a8 to 9bfbe9d Compare June 20, 2024 14:03
@hunjixin hunjixin changed the title Feat/cuda miner support feat/cuda miner support Jun 21, 2024
@hunjixin hunjixin force-pushed the feat/cuda_miner_support branch 3 times, most recently from e29ce8c to 221c765 Compare June 21, 2024 05:00
@hunjixin hunjixin force-pushed the feat/cuda_miner_support branch from 221c765 to dd560c9 Compare June 21, 2024 05:05
@hunjixin hunjixin changed the title feat/cuda miner support feat: cuda miner support Jun 21, 2024
@richardbremner
github action to perform github binary releases
9375723
@richardbremner
Merge branch 'feat/cuda_binary' into feat/cuda_miner_support
a01973b 
# Conflicts:
#	pkg/resourceprovider/cudaminer/keccak.cu
#	pkg/resourceprovider/cudaminer/keccak.ptx
#	pkg/resourceprovider/cudaworker.go
#	pkg/resourceprovider/others_os.go
@hunjixin hunjixin closed this Jun 21, 2024
@hunjixin hunjixin mentioned this pull request Jun 21, 2024
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hunjixin @richardbremner