Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to nv15 (actors v7, fvm v0.8) #1487

Merged
merged 53 commits into from
Jun 10, 2022
Merged

Upgrade to nv15 (actors v7, fvm v0.8) #1487

merged 53 commits into from
Jun 10, 2022

Conversation

lemmih
Copy link
Contributor

@lemmih lemmih commented Mar 21, 2022

Summary of changes
Changes introduced in this pull request:

  • Use actors v7.
  • Use FVM for tipset validation.
  • Sync all of nv15.

Reference issue to close (if applicable)

Other information and links

@codecov
Copy link

codecov bot commented Mar 21, 2022

Codecov Report

Merging #1487 (b35f3be) into main (ebed40a) will decrease coverage by 11.78%.
The diff coverage is 3.06%.

❗ Current head b35f3be differs from pull request most recent head e699682. Consider uploading reports for the commit e699682 to get more accurate results

@@             Coverage Diff             @@
##             main    #1487       +/-   ##
===========================================
- Coverage   44.42%   32.63%   -11.79%     
===========================================
  Files         310      319        +9     
  Lines       28112    28425      +313     
===========================================
- Hits        12488     9277     -3211     
- Misses      15624    19148     +3524     
Impacted Files Coverage Δ
blockchain/chain_sync/src/tipset_syncer.rs 0.00% <0.00%> (ø)
blockchain/state_manager/src/errors.rs 0.00% <0.00%> (ø)
blockchain/state_manager/src/lib.rs 1.84% <0.00%> (ø)
ipld/blockstore/src/lib.rs 48.71% <0.00%> (-10.66%) ⬇️
node/rpc/src/state_api.rs 0.00% <0.00%> (ø)
types/networks/src/lib.rs 58.33% <ø> (ø)
types/networks/src/mainnet/mod.rs 100.00% <ø> (ø)
utils/bitfield/src/lib.rs 75.00% <0.00%> (-10.14%) ⬇️
utils/statediff/src/lib.rs 0.00% <0.00%> (ø)
vm/actor_interface/src/builtin/account/mod.rs 0.00% <0.00%> (-30.00%) ⬇️
... and 81 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 57ea70e...e699682. Read the comment docs.

@lemmih lemmih changed the title Lemmih/builtin actors v7 Upgrade to nv15 (actors v7, fvm v0.8) Jun 9, 2022
@lemmih lemmih marked this pull request as ready for review June 10, 2022 15:35
@lemmih lemmih merged commit c209286 into main Jun 10, 2022
@lemmih lemmih deleted the lemmih/builtin-actors-v7 branch June 10, 2022 17:03
LesnyRumcajs added a commit that referenced this pull request Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants