!ciojfMuGYygKQdJjpD:matrix.org

Frontier

441 Members
Frontier technical discussion6 Servers

Load older messages


SenderMessageTime
13 Jan 2021
@jnaviasky:matrix.orgjnaviaskythe last one can probably be "master" now that the change has been merged19:02:20
@jnaviasky:matrix.orgjnaviaskyexcellent 🚀19:02:34
@jnaviasky:matrix.orgjnaviaskyRedacted or Malformed Event20:30:27
@jnaviasky:matrix.orgjnaviaskyjust found an issue with gas estimation in the EVM module: https://github.com/rust-blockchain/evm/issues/820:47:00
14 Jan 2021
@sorpaas:matrix.parity.ioWei TangAh thank you so much!15:17:01
@sorpaas:matrix.parity.ioWei 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
@sorpaas:matrix.parity.ioWei TangI'll still take a look and see if I'm wrong.15:19:30
@sorpaas:matrix.parity.ioWei TangBy 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
@sorpaas:matrix.parity.ioWei Tang The modular design of our evm crate allows us to swap out the gasometer quiet easily. 15:21:48
@joshyorndorff:matrix.orgJoshy OrndorffI did some brief thinking around that topic. Do you have any thoughts about it? https://github.com/rust-blockchain/evm/issues/615:44:21
@sorpaas:matrix.parity.ioWei 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
@sorpaas:matrix.parity.ioWei 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
@joshyorndorff:matrix.orgJoshy OrndorffYeah, 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
@xlchen:matrix.orgBryan 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:matrix.orgjnaviasky 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:matrix.orgjnaviaskyfor now I'll leave those commits in and just make a PR, we can discuss whether that's the best way to do it later21:00:49
@jnaviasky:matrix.orgjnaviaskyok posted it up https://github.com/paritytech/frontier/pull/26921:09:44
@notlesh:matrix.orgnotlesh 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
@sorpaas:matrix.parity.ioWei 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:matrix.orgjnaviaskydo we support validators selecting a minimum gas price on frontier?16:44:48
@jnaviasky:matrix.orgjnaviaskyI'm realizing I can send txs with gas price 0 and they'll get processed fine, no balance subtracted16:45:45
@jnaviasky:matrix.orgjnaviaskyfor now i'll implement a FixedGasPrice struct but this is an interesting point of divergence from eth proper16:48:42
@notlesh:matrix.orgnotlesh
In reply to @sorpaas:matrix.parity.io
Finishing it today. Sorry about the delay!
no worries, thanks much!
17:49:41
@sorpaas:matrix.parity.ioWei 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:matrix.orgjnaviasky 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:matrix.orgnasah joined the room.14:28:46
@mpagani:matrix.orgmpaganiAnyone else getting "Pool(TemporarilyBanned)" error on RPC calls with the recent changes on Frontier Template Node?15:26:45
@mpagani:matrix.orgmpaganiI just updated my node with the master branch, and it started giving this error on RPC transactions15:27:23
@mpagani:matrix.orgmpaganiOk, never mind. It was a bad merge on my side. Sry16:31:33
@mpagani:matrix.orgmpaganiBy the way, nice work with gas estimation. Just tested with my troublemaker contracts and it works like a charm16:37:32

There are no newer messages yet.


Back to Room List