-
Notifications
You must be signed in to change notification settings - Fork 7
add staking v2 precompile #64
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -27,10 +27,9 @@ In this tutorial you will learn how to interact with staking precompile in two w | |||||
|
||||||
```sh | ||||||
btcli subnet create --network ws://127.0.0.1:9944 | ||||||
btcli subnet register --network ws://127.0.0.1:9944 | ||||||
btcli subnet register --network ws://127.0.0.1:9944 | ||||||
``` | ||||||
|
||||||
|
||||||
3. Save the delegate hotkey address. You will use this in the staking pool use case below. | ||||||
|
||||||
4. Disable staking rate limits by setting `targetStakesPerInterval` to 1000. Follow these below steps: | ||||||
|
@@ -40,13 +39,17 @@ btcli subnet register --network ws://127.0.0.1:9944 | |||||
- Click on **Submit Transaction** at the bottom right. This will open the **authorize transaction** window. | ||||||
- On this **authorize transaction** window, make sure the **sign and submit** toggle is ON and click on the **Sign and Submit** on the bottom right. | ||||||
|
||||||
## Staking V1 and V2. | ||||||
|
||||||
There are two versions of staking precompile implemenation, V1 and V2. The contract address for V1 is `0x0000000000000000000000000000000000000801`. The address for V2 is `0x0000000000000000000000000000000000000805`. The V1 is deprecated, just being kept for compatiable with old interface. The major difference between V1 and V2 is, the staking amount is fetched from the msg.value in V1. Then precompile transfer the token back to caller. It is misleading and confuses the solidity developers. In V2 implementation, all amount parameters are defined as parameter of transaction. | ||||||
|
||||||
## Call the staking precompile from another smart contract (staking pool use case) | ||||||
|
||||||
In this interaction you will compile [`stake.sol`](https://github.com/opentensor/evm-bittensor/blob/main/solidity/stake.sol), a smart contract Solidity code and execute it on the subtensor EVM. This `stake.sol` will, in turn, call the staking precompile that is already deployed in the subtensor EVM. | ||||||
In this interaction you will compile [`stakeV2.sol`](https://github.com/opentensor/evm-bittensor/blob/main/solidity/stakeV2.sol), a smart contract Solidity code and execute it on the subtensor EVM. This `stakeV2.sol` will, in turn, call the staking precompile that is already deployed in the subtensor EVM. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
Before you proceed, familiarize yourself with the Solidity code of the [`stake.sol`](https://github.com/opentensor/evm-bittensor/blob/main/solidity/stake.sol) smart contract. | ||||||
Before you proceed, familiarize yourself with the Solidity code of the [`stakeV2.sol`](https://github.com/opentensor/evm-bittensor/blob/main/solidity/stakeV2.sol) smart contract. | ||||||
|
||||||
1. Copy the text of [`stake.sol`](https://github.com/opentensor/evm-bittensor/blob/main/solidity/stake.sol) contract to Remix IDE. | ||||||
1. Copy the text of [`stakeV2.sol`](https://github.com/opentensor/evm-bittensor/blob/main/solidity/stakeV2.sol) contract to Remix IDE. | ||||||
|
||||||
2. You will now convert your delegate hotkey ss58 from the above [Setup EVM localnet, subnet and delegate](#setup-evm-localnet-subnet-and-delegate) step into its corresponding public key. Use the [ss58.org](https://ss58.org/) site to obtain the public key for your delegate hotkey ss58. | ||||||
|
||||||
|
@@ -64,48 +67,11 @@ Before you proceed, familiarize yourself with the Solidity code of the [`stake.s | |||||
|
||||||
In this tutorial, you will interact directly with the staking precompile by using its ABI, and use your Metamask wallet as the source of TAO to stake. | ||||||
|
||||||
1. Copy this below ABI of staking precompile contract into Remix IDE as a new file: | ||||||
|
||||||
```json | ||||||
[ | ||||||
{ | ||||||
"inputs": [ | ||||||
{ | ||||||
"internalType": "bytes32", | ||||||
"name": "hotkey", | ||||||
"type": "bytes32" | ||||||
} | ||||||
], | ||||||
"name": "addStake", | ||||||
"outputs": [], | ||||||
"stateMutability": "payable", | ||||||
"type": "function" | ||||||
}, | ||||||
{ | ||||||
"inputs": [ | ||||||
{ | ||||||
"internalType": "bytes32", | ||||||
"name": "hotkey", | ||||||
"type": "bytes32" | ||||||
}, | ||||||
{ | ||||||
"internalType": "uint256", | ||||||
"name": "amount", | ||||||
"type": "uint256" | ||||||
} | ||||||
], | ||||||
"name": "removeStake", | ||||||
"outputs": [], | ||||||
"stateMutability": "payable", | ||||||
"type": "function" | ||||||
} | ||||||
] | ||||||
``` | ||||||
|
||||||
2. Copy staking precompile address `0x0000000000000000000000000000000000000801` to the **At Address** field in Remix IDE, and click **At Address** button. | ||||||
1. Copy this below ABI from https://github.com/opentensor/subtensor/blob/main/precompiles/src/solidity/stakingV2.abi into Remix IDE as a new file. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
3. Remix IDE will find the precompile at the precompile address on the subtensor EVM and show it in the list of deployed contracts. Expand the contract, then expand the `addStake` method, and paste the public key of your delegate hotkey into the `hotkey` field. Then click **transact** and wait for the transaction to be completed. | ||||||
2. Copy staking precompile address `0x0000000000000000000000000000000000000805` to the **At Address** field in Remix IDE, and click **At Address** button. | ||||||
|
||||||
3. Remix IDE will find the precompile at the precompile address on the subtensor EVM and show it in the list of deployed contracts. Expand the contract, then expand the `addStake` method, and paste the public key of your delegate hotkey into the `hotkey` field. Then click **transact** and wait for the transaction to be completed. | ||||||
|
||||||
4. Follow these steps to see that the stake record is updated in [Polkadot JS app](https://polkadot.js.org/apps/?rpc=ws%3A%2F%2F127.0.0.1%3A9944#/chainstate): | ||||||
|
||||||
|
@@ -114,3 +80,16 @@ In this tutorial, you will interact directly with the staking precompile by usin | |||||
3. Toggle **include option** OFF for the second parameter. | ||||||
4. Click the **+** button and find the new stake record. | ||||||
|
||||||
## Questions about Call the staking precompile from another smart contract. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
1. which one is the coldkey? | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
Since the precompile can't get the orginal caller, the precompile takes the contract's address as coldkey. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
2. is the token subtracted from contract or original caller if calling the addStake | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
The contract need to pay the token. so must guarantee the balance of contract is enough to call addStake. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
3. any security issue for the contract | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
Because the token for addStake is subtracted from contract. We must set the transaction as priviledged one. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
like the example in [the link will be available after a PR merged in subtensor side] | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
4. the unit of amount in addStake, removeStake | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
As the function parameter indicates, all these are aligned with the decimal of TAO | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
5. If transfer token to contract in msg.value, and trigger the contract to call precompile. should the function in contract do the decimal convert. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
Yes. the decimal in msg.value is 18. need conversion like `uint256 amount = msg.value / 1e9` | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.