Testnet hard fork: Allium PoW algorithm #39
Conversation
ryan-shaw
changed the title
src/chainparams.h
Outdated
| @@ -78,6 +78,17 @@ class CChainParams | |||
| const CCheckpointData& Checkpoints() const { return checkpointData; } | |||
| const ChainTxData& TxData() const { return chainTxData; } | |||
| void UpdateVersionBitsParameters(Consensus::DeploymentPos d, int64_t nStartTime, int64_t nTimeout); | |||
| int SwitchAllium(int nHeight) const { | |||
There was a problem hiding this comment.
For readability sake, function could be reduced down to:
return strNetworkID == CBaseChainParams::TESTNET && nHeight > 10
There was a problem hiding this comment.
I'd rather suggest changing this to a "GetPoWAlgo" with fall through cases. Just for future-proofing. Also, rather than returning int's, you could define a global int-based enum. Would make it easier to read any comparisons in the rest of the code.
src/chainparams.h
Outdated
| @@ -78,6 +78,17 @@ class CChainParams | |||
| const CCheckpointData& Checkpoints() const { return checkpointData; } | |||
| const ChainTxData& TxData() const { return chainTxData; } | |||
| void UpdateVersionBitsParameters(Consensus::DeploymentPos d, int64_t nStartTime, int64_t nTimeout); | |||
| int SwitchAllium(int nHeight) const { | |||
There was a problem hiding this comment.
I'd rather suggest changing this to a "GetPoWAlgo" with fall through cases. Just for future-proofing. Also, rather than returning int's, you could define a global int-based enum. Would make it easier to read any comparisons in the rest of the code.
src/crypto/allium/allium.c
Outdated
| // hash[i], hash[i+1], hash[i+2], hash[i+3]); | ||
| // } | ||
| // return buf; | ||
| // } |
There was a problem hiding this comment.
Commented debug should be removed. (it's taken from utils either way)
|
Refactored algo selection method |
In light of ASICs being able to mine on Scrypt-N a new proof of work algorithm is needed to remove these unwanted guest on the network. Allium is a custom variant of the Lyra2 proof of algorithm that is already tested greatly in other coins.
Because of the custom nature of the PoW algorithm a fork of miners and pool software is needed:
Pool software:
CPU miners:
GPU miners:
TODO: