| 13 Jan 2021 |
 jnaviasky | the last one can probably be "master" now that the change has been merged | 19:02:20 |
| jnaviasky | excellent 🚀 | 19:02:34 |
| jnaviasky | Redacted or Malformed Event | 20:30:27 |
| jnaviasky | just found an issue with gas estimation in the EVM module: https://github.com/rust-blockchain/evm/issues/8 | 20:47:00 |
| 14 Jan 2021 |
 Wei Tang | Ah thank you so much! | 15:17:01 |
| Wei Tang | Hmm yeah.. I'll try to see if I can find any solutions, but otherwise this might mean that the method of doing evm estimation internally in the evm crate will never work, and we have to resort to binary-search only. | 15:19:19 |
| Wei Tang | I'll still take a look and see if I'm wrong. | 15:19:30 |
| Wei Tang | By the way, given Ethereum community's nobody-should-make-assumptions-about-gas-cost statement for dapp developers, it may also be safe if we completely ditch the Ethereum gasometer and switch to one that is more suitable for Substrate costs. | 15:21:04 |
| Wei Tang | The modular design of our evm crate allows us to swap out the gasometer quiet easily. | 15:21:48 |
 Joshy Orndorff | I did some brief thinking around that topic. Do you have any thoughts about it? https://github.com/rust-blockchain/evm/issues/6 | 15:44:21 |
| Wei Tang | In reply to @joshyorndorff:matrix.org
I did some brief thinking around that topic. Do you have any thoughts about it? https://github.com/rust-blockchain/evm/issues/6 I don't really object that, but my suggestion is that we find a concrete use-case before doing this generalization. Parity Ethereum / Open Ethereum had this generalization already for a long time, but it ended up not being used (with UnsignedUnits always fixed to a single value in the whole code). | 16:25:17 |
| Wei Tang | I like the idea of ComputeAndStorageCosts and I think that might work well (even if we don't do the UnsignedUnits generalization). I do want to note though -- that some part of Ethereum's gasometer is really weird and making it to fit the model of ComputeAndStorageCosts may be impossible. In that case my suggestion is that we diverge from what Ethereum is doing. | 16:27:17 |
| Joshy Orndorff | Yeah, all I really want is separate compute and storage cost tracking. My concrete use case is that we want to run a substrate chain hosting the EVM (like many people here). In Substrate we limit execution time using Weight. But in EVM we have only gas. I want to seperate the storage and compute so that we can more accurately count the actual execution time of the EVM. | 18:41:31 |
 Bryan Chen | Laminar & Acala | We have actually already started building a storage meter. Will share it here once I got it working. | 19:08:30 |
| jnaviasky | got a question for Wei Tang about the best way to go about doing this -- i made some changes to https://github.com/paritytech/frontier/pull/252 to correct what i saw as some issues in its implementation, was wondering whether it makes sense to make an entirely new PR for it, or whether to make a PR against that PR branch? | 20:58:07 |
| jnaviasky | for now I'll leave those commits in and just make a PR, we can discuss whether that's the best way to do it later | 21:00:49 |
| jnaviasky | ok posted it up https://github.com/paritytech/frontier/pull/269 | 21:09:44 |
 notlesh | sorpaas: is there any chance evm could be updated with this bump to primitive-types? https://github.com/rust-blockchain/evm/pull/7 | 22:14:49 |
| 15 Jan 2021 |
| Wei Tang | In reply to @notlesh:matrix.org
sorpaas: is there any chance evm could be updated with this bump to primitive-types? https://github.com/rust-blockchain/evm/pull/7 Finishing it today. Sorry about the delay! | 11:21:50 |
| jnaviasky | do we support validators selecting a minimum gas price on frontier? | 16:44:48 |
| jnaviasky | I'm realizing I can send txs with gas price 0 and they'll get processed fine, no balance subtracted | 16:45:45 |
| jnaviasky | for now i'll implement a FixedGasPrice struct but this is an interesting point of divergence from eth proper | 16:48:42 |
| notlesh | In reply to @sorpaas:matrix.parity.io
Finishing it today. Sorry about the delay! no worries, thanks much! | 17:49:41 |
| Wei Tang | In reply to @jnaviasky:matrix.org
do we support validators selecting a minimum gas price on frontier? There's the minimal gas price trait in the config. We can custom the gas price logic using that. | 18:01:00 |
| jnaviasky | ideally we could pass a flag like eth uses --gasprice, but i guess that would be more of a node thing than a frontier thing | 18:23:11 |
| 18 Jan 2021 |
|  nasah joined the room. | 14:28:46 |
 mpagani | Anyone else getting "Pool(TemporarilyBanned)" error on RPC calls with the recent changes on Frontier Template Node? | 15:26:45 |
| mpagani | I just updated my node with the master branch, and it started giving this error on RPC transactions | 15:27:23 |
| mpagani | Ok, never mind. It was a bad merge on my side. Sry | 16:31:33 |
| mpagani | By the way, nice work with gas estimation. Just tested with my troublemaker contracts and it works like a charm | 16:37:32 |
