Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Nov 19 16:08
    Elnaril opened #1964
  • Nov 18 18:04
    Elnaril commented #1942
  • Nov 17 09:06
    Peppece commented #1937
  • Nov 17 09:01
    Peppece commented #1937
  • Nov 06 11:20
    gsalgado commented #1961
  • Nov 06 10:13
    gsalgado opened #1963
  • Nov 06 10:12
    gsalgado opened #1962
  • Nov 06 10:09
    gsalgado opened #1961
  • Nov 03 17:41
    Peppece review_requested #1937
  • Nov 03 17:24
    Peppece synchronize #1937
  • Nov 03 17:19
    Peppece synchronize #1937
  • Nov 03 17:11
    Peppece synchronize #1937
  • Oct 29 18:45
    Pet3ris commented #1960
  • Oct 29 18:44
    Pet3ris commented #1960
  • Oct 29 16:37
    carver commented #1960
  • Oct 29 13:04
    Pet3ris opened #1960
  • Oct 25 08:34
    Peppece edited #1937
  • Oct 25 08:01
    Peppece review_requested #1937
  • Oct 25 08:00
    Peppece edited #1937
  • Oct 25 07:46
    Peppece synchronize #1937
Just some guy
@fubuloubu
(which uses py-evm 0.2.0a43+ under the hood)
Istanbul was only introduced in eth-tester v0.2b3 I believe
Voith Mascarenhas
@voith
Hmmm, yeah. But istanbul could break some tests, specially ethereum/EIPs#2200
Just some guy
@fubuloubu
I replaced in our test cases SELFDESTRUCT with a simple send that replicates the same result, and it passes
Voith Mascarenhas
@voith
:+1:
Just some guy
@fubuloubu
btw @carver the CHANGELOG for eth-tester still says v0.2.0b3 is unreleased
Noel Maersk
@veox
We should probably change 7_000_000 to 5_000_000 (same as Constantinople/Petersburg) on this line, if there is to be no difficulty bomb reset for Istanbul. (And looks like there won't be.) (EDIT: Or use the petersburg function altogether.)
I've done some slackjob calculations, and (given no changes in net-wide hashrate, a big given) the bomb won't make 10% of the difficulty for ~500 more days.
A tad too optimistic for my taste, since unexpected delays are always possible (as Constantinople/Petersburg showed), but I'll refrain from suggesting in ACD a "reset" should be done preemptively.
Christoph Burgdorf
@cburgdorf
Just merged Clique Consensus PR ethereum/py-evm#1855 Trinity side should follow soon.
Noel Maersk
@veox
QQ: Does newsfragments support multiple lines?..
Noel Maersk
@veox
As in, there are two noteworthy changes in PR #1878, that should both be listed...
OK, towncrier --draft is my friend. ^_^
Noel Maersk
@veox
Is there a "common repo" for the boilerplate newsfragments/README.md file? Turns out fragments from the misc category don't get their contents displayed, they just get the numbers listed in a "Miscellaneous internal changes" section.
Christoph Burgdorf
@cburgdorf
@veox if those are separate categories (e.g performance and feature) then you can create two individual files. If those are of the same category then I'm unaware of differentiating them apart from line breaks.
@veox eventually it will make it into the ethereum-python-project-template ethereum/ethereum-python-project-template#28 but we wanted to wait for the next Trinity release to see if there are any further tweaks to the release script needed.
Noel Maersk
@veox
Thanks, looks like just the thing.
Jason Carver
@carver

Turns out fragments from the misc category don't get their contents displayed, they just get the numbers listed in a "Miscellaneous internal changes" section.

Yeah, as far as I know, someone did this intentionally. I added an "internal" section to trinity that shows towncrier content here: https://github.com/ethereum/trinity/blob/62a85b1d3c89908c37fd62d9c4370dc8e3b139e2/pyproject.toml#L34-L37 -- the idea is that it's only of interest to contributors rather than users. Then misc would be for trivial changes, like maybe changing flake8 rules and updating the repo. @veox

Jason Carver
@carver
I merged in the #1858 changes (in #1871 ) and am starting in on a release...
@/all
Voith Mascarenhas
@voith
:tada:
Just some guy
@fubuloubu
Thanks for the fix to ethereum/py-evm#1865
Jason Carver
@carver
:+1:
Jacques Wagener
@jacqueswww
:clap:
Chris Calderon
@SerpentChris
If I want to make a chain to use for testing a smart contract, can I use the Chain class instead of the MiningChain class?
Christoph Burgdorf
@cburgdorf
@SerpentChris You need the MiningChain class when you want to build blocks incrementally instead of importing blocks in one go.
Jason Carver
@carver
For testing, you might be interested in using web3.py with eth-tester (which uses py-evm under the hood)
Just some guy
@fubuloubu
is Python 3.8 in the works for this project and eth-tester?
Jason Carver
@carver
I don't think there should be any problems w/ running it in py3.8. Do you know of anything? Obviously we'll want to add tests at some point, but it's not an active project that I know of @fubuloubu
Jason Carver
@carver
nm, I see in ethereum/vyper#1678 that there's some problem with the blake2b library
Just some guy
@fubuloubu
We were trying to upgrade, but I think it's just been premature. Is it just me, or has 3.7/3.8 moved a lot faster than previous release?
Monir Shaker
@Moniroaf
i have some problems to compile the app.py i conudn't call the contract from the file sol ?!!
pinnaculum
@pinnaculum
congrats on py-evm this is huge
Noel Maersk
@veox

18 days later, ACD is noisy because of the difficulty bomb. :/

TL;DR: There might be an out-of-schedule call on Friday.

(I suggest you wonderful busy folk don't waste time on reading it all.)
Jason Carver
@carver
whoops, thanks for pointing it out (twice now!)
Noel Maersk
@veox
Heh, that's not what I meant by the comment, but thanks for fixing. :)
Jason Carver
@carver
:p
Griffin Ichiba Hotchkiss
@gichiba
Happy thanksgiving, americans! I'm looking to be pointed in the right direction with how to play with py-evm... I want to do essentially what is done in this exercise: https://easythereentropy.wordpress.com/2014/06/04/understanding-the-ethereum-trie/ -- but using py-evm instead of pythereum, and using the nice boilerplate project template that you have in the cookbooks. My question is where in the py-evm API or source would I find the logic that does the same thing trie.py does in pyethereum? Or, perhaps I'm totally going about this the wrong way, in which case my question is even more general: What's a good starting place for simple DB/State/Trie understanding of the evm?
Jason Carver
@carver
Yeah, if you just want to play around with tries, check out https://github.com/ethereum/py-trie @gichiba
I just noticed an "early release" warning on the README of py-trie. That could be removed now. It is the library that py-evm uses, and has gone through a solid amount of testing at this point. It could be optimized a lot more, but I'm not generally concerned about correctness, at least for evm-specific configurations (like always using 32-byte keys).
Griffin Ichiba Hotchkiss
@gichiba
Thanks @carver !
Jason Carver
@carver
I'm preparing py-evm for the Istanbul fix release today. The previous release incorrectly included an ice age delay.
Jason Carver
@carver
@/all py-evm with the fixed Istanbul (and a couple other things) is released:
https://py-evm.readthedocs.io/en/latest/release_notes.html#py-evm-0-3-0-alpha-9-2019-12-02
Noel Maersk
@veox

I have a feeling someone'll have to take over PR #1885 from me, or provide a lot of guidance.

The current lint failures are all for improper definitions/returns in low-level API classes: AccountDatabaseAPI, OpcodeAPI, VirtualMachineAPI; and some related abstract classes.

Take a look at this CI job most errors are an either-or choice, and I don't know which one is "correct".

There's also

eth/chains/base.py:250: error: Unexpected keyword argument "chain_context" for "VirtualMachineAPI"
eth/abc.py:2270: note: "VirtualMachineAPI" defined here

which means something unexpected may be happening.

Jason Carver
@carver
I'm on mobile now, but if you cc me on the PR, I'll take a look
Noel Maersk
@veox
@carver I went AFK immediately after posting y-day, sorry. Maybe it was a little too panicky. I'll cc you in a sec.
Jason Carver
@carver
@/all critical Istanbul bugfix release: https://py-evm.readthedocs.io/en/latest/release_notes.html#py-evm-0-3-0-alpha-10-2019-12-09
Trinity release coming shortly
Kjetil Vaagen
@KjetilVaa
Does anyone have any experience with Clique consensus who can help?
Noel Maersk
@veox
@KjetilVaa Very little. Anything specific?