Ignacio Hagopian

Results 70 comments of Ignacio Hagopian

Yes, at a minimum any protocol change that would cause a divergence in the state of non updated validators.

Adding a note: we should probably move the `config.json` and probably all the state out of sub-folders of the repo to avoid conflicts when updating.

For PR readers, extra commands where added via https://github.com/gballet/go-ethereum/pull/435

@gballet, the PR is still a draft and there're some red CI. Want me to review anyway as requested, or should I wait until is opened?

We're interested in this feature. It would make packing bigger-than-a-sectors DAGs in sector-sized deals much simpler since we don't have to deal with "complete-subdags" constraints. So, just pack the max...

I'm not yet convinced about the utility of this change for the following reasons: - The EVM execution today isn't a bottleneck in verifiers. - In the Verkle fork, we...

@chfast, I applied your suggestions if you'd like to take another look. Cycles remained the same for everything, but now both MSM make the SP1 RISCV emulator crash :)

@kevaundray, @jochem-brouwer, we have this `STOP` case already covered in my warm-CALL-like test. See [here](https://github.com/ethereum/execution-spec-tests/blob/jsign-0-param-opcodes/tests/zkevm/test_worst_stateful_opcodes.py#L151). What I do for warm-CALL-like is exactly that. Call a contract that starts with `STOP`,...

@jochem-brouwer, thanks for the idea :) ~But I think we should probably do `gas=0 if frame_terminator=Op.INVALID else Op.GAS`, if we don't do that we would be missing actually "executing" the...