search

near / nearcore

Reference client for NEAR Protocol
https://near.org
GNU General Public License v3.0
2.3k stars 600 forks source link

fix test_full_estimator #11705

Closed wacban closed 2 days ago

wacban commented 3 days ago

The test_full_estimator is disabled due to errors coming from the congestion control stack. Likely the congestion info is not correctly bootstrapped. Fix this issue and re-enable the test.

wacban commented 3 days ago

JFYI @jakmeier @nagisa

jakmeier commented 2 days ago

I have the fix caused by congestion control changes. We need to update the apply_state between chunks with the updated congestion info and it works again.

But I'm still working on further problems, caused by new limits that change how some estimations should work, since we can no longer fill a block with X * 1000 data receipts with a 100KB size each and expect it to finish within a single block.