cardano-miners / fortuna

All the "greatness" of proof of work, now in smart contract form
https://minefortuna.com
Apache License 2.0
52 stars 23 forks source link

Rust Miner Arch Improvements #32

Closed MitchTurner closed 1 year ago

MitchTurner commented 1 year ago

I kept thinking about this so I figured I'd write it up and put it out there.

I kept the worker manager as a trait as well because I keep hearing about doing GPU hashing. We could develop a CPU manager for now and then someone could upgrade it to GPU once we figure it out. The abstraction makes that easier.

MitchTurner commented 1 year ago

It's possible that more logic could be pushed into the Miner::run() function, maybe by letting the manager tell you how many threads to create or something, but this is the general idea.

rvcas commented 1 year ago

yea this is great, I'm down to move these changes into src