filecoin-project / specs-actors

DEPRECATED Specification of builtin actors, in the form of executable code.

Other

86 stars 102 forks source link

Feat/market dobe rand #1445

Closed ZenGround0 closed 3 years ago

ZenGround0 commented 3 years ago

Another fix for #1435. This avoids the problem of increasing wait time between deal start and first processing by up to another day by choosing between next and current day based on offset. It should remove all grinding possibilities because processing epoch is now assigned predictably but unbiasedly.

Tests still need to be fixed -- currently market tests everywhere assume the ability to set processing epoch which makes this a long fix

wadealexc commented 3 years ago

LGTM!

codecov-commenter commented 3 years ago

Codecov Report

Merging #1445 (f863c31) into master (596cff3) will increase coverage by 0.0%. The diff coverage is 94.7%.

@@          Coverage Diff           @@
##           master   #1445   +/-   ##
======================================
  Coverage    70.2%   70.2%           
======================================
  Files          72      72           
  Lines        7790    7791    +1     
======================================
+ Hits         5469    5472    +3     
+ Misses       1438    1437    -1     
+ Partials      883     882    -1