Skip to content
This repository was archived by the owner on Nov 6, 2020. It is now read-only.

Update ProgPoW to 0.9.3#11407

Merged
andresilva merged 1 commit into
openethereum:masterfrom
ethercore:progpow
Jan 24, 2020
Merged

Update ProgPoW to 0.9.3#11407
andresilva merged 1 commit into
openethereum:masterfrom
ethercore:progpow

Conversation

@EtherCoreAdmin
Copy link
Copy Markdown
Contributor

@EtherCoreAdmin EtherCoreAdmin commented Jan 24, 2020

0.9.2 version is deprecated due to efficiency

See 0.9.3 specification on https://github.com/ethereum/EIPs/blob/master/EIPS/eip-1057.md#specification

0.9.2 version is deprecated due to efficiency
@parity-cla-bot
Copy link
Copy Markdown

It looks like @ethcoreorg signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@ordian ordian requested a review from andresilva January 24, 2020 15:06
@ordian ordian added A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust. labels Jan 24, 2020
Copy link
Copy Markdown
Contributor

@andresilva andresilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@andresilva
Copy link
Copy Markdown
Contributor

@ethcoreorg how did you update the values for the test vectors? Did you confirm we're getting the same results as other implementations?

@EtherCoreAdmin
Copy link
Copy Markdown
Contributor Author

@EtherCoreAdmin
Copy link
Copy Markdown
Contributor Author

I've also confirmed that the test vector works for go-ethereum implementation also holiman/go-ethereum#4

@andresilva
Copy link
Copy Markdown
Contributor

@ethcoreorg I saw the EIP link you posted and checked the changes in constants but didn't notice the test vectors. Thanks! :)

@andresilva andresilva merged commit 108daf1 into openethereum:master Jan 24, 2020
@ordian ordian added the B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. label Jan 27, 2020
s3krit pushed a commit that referenced this pull request Jan 27, 2020
0.9.2 version is deprecated due to efficiency
dvdplm added a commit that referenced this pull request Jan 29, 2020
…pstream

* master:
  Add POSDAO transition and malice report queue. (#11245)
  update master/nightly version: v2.8.0 (#11419)
  ethcore/res: remove morden testnet (#11392)
  fix: export hardcoded sync format (#11416)
  update hardcoded headers: mainnet and ropsten (#11414)
  AuthorityEngine: Minor cleanups. (#11408)
  Update POA bootnodes (#11411)
  Add EtherCore support (#11402)
  verification: fix race same block + misc (#11400)
  Update ProgPoW to 0.9.3 (#11407)
  update classic testnet bootnodes (#11398)
  dependencies: bump `derive_more v0.99` (#11405)
  engine error: remove faulty/unused `From` (#11404)
  Switching to stable-track (#11377)
  ethcore/res: fix ethereum classic chainspec blake2_f activation block num (#11391)
  Update copyright notice 2020 (#11386)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A8-looksgood 🦄 Pull request is reviewed well. B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. M4-core ⛓ Core client code / Rust.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants