Teep pricing changes

[Stebalien]

No description provided.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 75.95%. Comparing base (525faea) to head (61337f1).

Files with missing lines	Patch %	Lines
fvm/src/gas/price_list.rs	50.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2103      +/-   ##
==========================================
- Coverage   75.96%   75.95%   -0.01%     
==========================================
  Files         154      154              
  Lines       15771    15772       +1     
==========================================
  Hits        11980    11980              
- Misses       3791     3792       +1     

Files with missing lines	Coverage Δ
fvm/src/gas/price_list.rs	77.47% <50.00%> (-0.16%)	⬇️

[rvagg]

we should probably hold off on merging this for now eh?

[Stebalien]

we should probably hold off on merging this for now eh?

Yeah, probably.

[Stebalien]

I also made a bunch of other changes in https://github.com/filecoin-project/ref-fvm/pull/1854/files and I want to make sure we're not missing something.

[Stebalien]

Ok, I made two changes there:

1. I charged less for batches, assuming some amount of parallelism.
2. I changed some functions around, but that had no real impact on functionality.

[BigLep]

we should probably hold off on merging this for now eh?

Yeah, probably.

When do we plan to merge this?

[rvagg]

If FIP-0100 gets accepted this gets merged. If it doesn't get accepted we're going to need to have another talk about this.

[BigLep]

@rvagg : I assume we're good to merge this now that FIP-0100 is effectively accepted (and will be accepted by EI=OD 2025-03-14).

I assume there is communication we need to do with consumers like Venus and Forest to make sure they update?

[rvagg]

We have #2107 too, but when we cut a release they'll need to update.