status-im / nim-eth-verkle

Verkle tree implementation
Apache License 2.0
8 stars 3 forks source link

Update codebase with the new constantine version #26

Closed advaita-saha closed 4 months ago

advaita-saha commented 5 months ago

The PR https://github.com/mratsim/constantine/pull/392 in constantine introduces few generalisations & few type definition changes which would require changes in this codebase for Verkle Tries

mratsim commented 5 months ago

I have at least another PR as a follow-up for IPA. And probably a third one for cosmetic, for example the Point and Field names are conflicting with other names in Constantine and will be renamed EthVerklePoint and EthVerkleField for disambiguation.

One thing I have to look into there is whether you need the affine, projective or both for the verkle tries.