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

feat: fvm: FVM integration #8332

Merged
merged 38 commits into from
Mar 17, 2022
Merged

feat: fvm: FVM integration #8332

merged 38 commits into from
Mar 17, 2022

Conversation

arajasek
Copy link
Contributor

Backports #8293, see description there.

arajasek and others added 30 commits February 4, 2022 12:46
1. Move lock, loading, etc into GetFilVested.
2. Call it directly when creating the FVM.
3. Detach GetFilLocked from state manager.

Really, this just makes it a bit easier to reason about this code.
…work-version

feat: cli: set current network version from params
Updates the FVM
This is what the lotus VM does.
Use either Lotus VM or FVM consistently
@arajasek arajasek requested a review from a team as a code owner March 16, 2022 19:58
@codecov
Copy link

codecov bot commented Mar 17, 2022

Codecov Report

Merging #8332 (2ed2ee1) into release/v1.15.1 (114cb4e) will decrease coverage by 0.06%.
The diff coverage is 26.33%.

Impacted file tree graph

@@                 Coverage Diff                 @@
##           release/v1.15.1    #8332      +/-   ##
===================================================
- Coverage            40.44%   40.37%   -0.07%     
===================================================
  Files                  679      681       +2     
  Lines                74118    74294     +176     
===================================================
+ Hits                 29975    29995      +20     
- Misses               38901    39049     +148     
- Partials              5242     5250       +8     
Impacted Files Coverage Δ
chain/stmgr/utils.go 21.84% <0.00%> (-0.76%) ⬇️
chain/vm/fvm.go 0.00% <0.00%> (ø)
chain/vm/gas.go 78.57% <ø> (ø)
chain/vm/gas_v0.go 92.94% <ø> (ø)
.../lotus-sim/simulation/blockbuilder/blockbuilder.go 0.00% <0.00%> (ø)
node/builder_chain.go 100.00% <ø> (ø)
chain/consensus/filcns/compute_state.go 71.59% <25.00%> (-2.53%) ⬇️
chain/vm/vmi.go 50.00% <50.00%> (ø)
chain/stmgr/call.go 72.43% <64.28%> (-2.72%) ⬇️
chain/stmgr/supply.go 63.77% <66.66%> (ø)
... and 28 more

@arajasek arajasek merged commit ff244fb into release/v1.15.1 Mar 17, 2022
@arajasek arajasek deleted the feat/fvm branch March 17, 2022 16:20
@jennijuju
Copy link
Member

bad pr title lolol

@jennijuju jennijuju changed the title FVM integration feat: fvm: FVM integration Mar 17, 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