Conversation
|
@roman-khimov Please check it. |
| } | ||
|
|
||
| [ContractMethod(0_01000000, CallFlags.ReadStates)] | ||
| private long GetMinimumDeploymentFee(StoreView snapshot) |
There was a problem hiding this comment.
And maybe it's better to have this value in Policy contract.
There was a problem hiding this comment.
I'm not sure. This is a parameter related to contract deployment, and maybe it should be placed here.
There was a problem hiding this comment.
I think like @roman-khimov , it's related to the environment. But also it's ok too
| public static class NativeContractExtensions | ||
| { | ||
| public static ContractState DeployContract(this StoreView snapshot, UInt160 sender, byte[] nefFile, byte[] manifest, long gas = ApplicationEngine.TestModeGas) | ||
| public static ContractState DeployContract(this StoreView snapshot, UInt160 sender, byte[] nefFile, byte[] manifest, long gas = 200_00000000) |
There was a problem hiding this comment.
it's 100?
| public static ContractState DeployContract(this StoreView snapshot, UInt160 sender, byte[] nefFile, byte[] manifest, long gas = 200_00000000) | |
| public static ContractState DeployContract(this StoreView snapshot, UInt160 sender, byte[] nefFile, byte[] manifest, long gas = 100_00000000) |
| } | ||
|
|
||
| [ContractMethod(0_01000000, CallFlags.ReadStates)] | ||
| private long GetMinimumDeploymentFee(StoreView snapshot) |
There was a problem hiding this comment.
I think like @roman-khimov , it's related to the environment. But also it's ok too
|
Merge? |
|
@Tommo-L proposed to reduce the min fee to 1, what do you think? |
shargon
left a comment
There was a problem hiding this comment.
we can adjust the value later
vncoelho
left a comment
There was a problem hiding this comment.
It should be on the Police too.
vncoelho
left a comment
There was a problem hiding this comment.
But if it is just 1 GAS then I think it would be ok here.
|
I see we can adjust it later, but it needs the multi-signatures of committee. And 100GAS is still higher than Ethereum, which is about 0.04 ~ 0.2 ETH(~ 16GAS - 80GAS) for a simple ERC20 contract. |
|
I think this way will be better. DeployFee= baseFee + Factor * engine.StoragePrice * (nefFile.Length + manifest.Length)Like: Perhaps we can do a test with the Nep17 template. |
|
We should not adjust |
|
The size of NEP17 template is about 4KB, nef 2KB, manifest 2KB, which needs about 4GAS in the old way. |
|
I like the old way. If for malicious attack concern, we could increase that to some extent but should not be so high. |
Close #2001