filecoin-project / cpp-filecoin

C++17 implementation of Filecoin
https://filecoin.io/
GNU General Public License v3.0
126 stars 36 forks source link

Disable Estimator #667

Closed ortyomka closed 2 years ago

ortyomka commented 2 years ago

Signed-off-by: ortyomka iurin.art@gmail.com

Description of the Change

Disable Estimator, due to it has broken usual scheduling(Overloading most powerful workers and underloading weak workers)

Benefits

Possible Drawbacks

Usage Examples or Tests [optional]

Alternate Designs [optional]

codecov[bot] commented 2 years ago

Codecov Report

Merging #667 (366eff7) into master (f1c2c44) will decrease coverage by 0.17%. The diff coverage is 35.10%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #667      +/-   ##
==========================================
- Coverage   45.28%   45.11%   -0.18%     
==========================================
  Files         740      742       +2     
  Lines       33724    33935     +211     
  Branches    18765    18899     +134     
==========================================
+ Hits        15273    15310      +37     
- Misses      14015    14138     +123     
- Partials     4436     4487      +51     
Impacted Files Coverage Δ
core/miner/main/main.cpp 0.00% <ø> (ø)
core/sector_storage/impl/scheduler_impl.hpp 100.00% <ø> (ø)
core/sector_storage/impl/new_scheduler_impl.cpp 30.59% <30.59%> (ø)
core/sector_storage/impl/scheduler_impl.cpp 71.29% <66.66%> (-0.96%) :arrow_down:
core/sector_storage/impl/new_scheduler_impl.hpp 81.81% <81.81%> (ø)
core/primitives/resources/resources.hpp 11.32% <0.00%> (-33.02%) :arrow_down:
...orage/ipfs/graphsync/impl/network/peer_context.cpp 52.80% <0.00%> (+1.59%) :arrow_up:
...ore/storage/ipfs/graphsync/impl/graphsync_impl.cpp 65.16% <0.00%> (+2.24%) :arrow_up:
...ore/storage/ipfs/graphsync/impl/local_requests.cpp 45.45% <0.00%> (+5.68%) :arrow_up: