-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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: f3: update go-f3 to 0.2.0 #12390
Conversation
Includes: - fix for excessive bandwidth usage - significant performance improvements - minor consensus fixes Signed-off-by: Jakub Sztandera <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update the PR title to match https:/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update the PR title to match https:/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions
Signed-off-by: Jakub Sztandera <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update the PR title to match https:/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update the PR title to match https:/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions
🤷 no idea why it is is still angry |
PR title now matches the required format.
That is different than what is in the doc: https:/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions |
PR title now matches the required format.
It originally said it in the comment like so: #12340 (review) - but we missed brining that over to the https:/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions. Will add |
* Update go-f3 to 0.2.0 Includes: - fix for excessive bandwidth usage - significant performance improvements - minor consensus fixes Signed-off-by: Jakub Sztandera <[email protected]> * add changelog Signed-off-by: Jakub Sztandera <[email protected]> * chore(f3): update to final released version --------- Signed-off-by: Jakub Sztandera <[email protected]> Co-authored-by: Steven Allen <[email protected]>
* Update go-f3 to 0.2.0 Includes: - fix for excessive bandwidth usage - significant performance improvements - minor consensus fixes Signed-off-by: Jakub Sztandera <[email protected]> * add changelog Signed-off-by: Jakub Sztandera <[email protected]> * chore(f3): update to final released version --------- Signed-off-by: Jakub Sztandera <[email protected]> Co-authored-by: Steven Allen <[email protected]>
…lease) (#12400) * fix: lotus-miner: remove provecommit1 method (#12251) * remove provecommit1 * add changelog * update precommit and commit params * fix lint error * fix commit params * dep: f3: Update go-f3 to 0.0.6, enable it on mainnet (#12295) * Update go-f3 to 0.0.6 Signed-off-by: Jakub Sztandera <[email protected]> * Enable F3 in passive configuration in mainnet config Signed-off-by: Jakub Sztandera <[email protected]> * Add changelog Signed-off-by: Jakub Sztandera <[email protected]> * add new butterfly assets --------- Signed-off-by: Jakub Sztandera <[email protected]> Co-authored-by: Jennifer Wang <[email protected]> * retract v1.28.0 * update v1.28.0 changelog and add v1.28.1 * Update CHANGELOG.md * wip - update f3 * don't convert bigint type We now use the same one in GPBFT. * update docs * fix wrong param name * update butterfy assets * update go-f3 * update changelog * update version * fix typo * Update CHANGELOG.md Co-authored-by: Steven Allen <[email protected]> * Update CHANGELOG.md Co-authored-by: Rod Vagg <[email protected]> * Update CHANGELOG.md Co-authored-by: Rod Vagg <[email protected]> * apply f3 patch * chore: bump versions and make gen/docsgen-cli chore: bump versions and make gen/docsgen-cli * chore: update v1.28.2 changelog chore: update v1.282. changelog * feat: f3: update go-f3 to 0.2.0 (#12390) * Update go-f3 to 0.2.0 Includes: - fix for excessive bandwidth usage - significant performance improvements - minor consensus fixes Signed-off-by: Jakub Sztandera <[email protected]> * add changelog Signed-off-by: Jakub Sztandera <[email protected]> * chore(f3): update to final released version --------- Signed-off-by: Jakub Sztandera <[email protected]> Co-authored-by: Steven Allen <[email protected]> * fix!: sealer: handle initialisation error without panic storage/pipeline.NewPreCommitBatcher and storage/pipeline.New now have an additional error return to deal with errors arising from fetching the sealing config. * add breaking API upgrade warning to the ChangeLog * NewCommitBatcher now has an additional error return to deal with errors arising from fetching the sealing config. * fix: miner: Fix DDO pledge math (#12341) * Power is units of Space * Time so multiply by deal duration * fix: miner: Fix DDO pledge math * appease the changelog checker * Fix gen --------- Co-authored-by: zenground0 <[email protected]> * chore: fix lint error - Updated the logging statement in `testOutOfGasError` to correctly reference `build.BlockGasLimit` instead of `buildconstants.BlockGasLimit`. * fix: update changelog to reference bandwidth issue ticket fix: update changelog to reference bandwidth issue ticket * Update CHANGELOG.md Co-authored-by: Steve Loeppky <[email protected]> * Update CHANGELOG.md * chore: make gen and make docsgen-cli Run `make gen` and `make docsgen-cli` --------- Signed-off-by: Jakub Sztandera <[email protected]> Co-authored-by: LexLuthr <[email protected]> Co-authored-by: Jakub Sztandera <[email protected]> Co-authored-by: Jennifer Wang <[email protected]> Co-authored-by: Jiaying Wang <[email protected]> Co-authored-by: Steven Allen <[email protected]> Co-authored-by: Rod Vagg <[email protected]> Co-authored-by: aarshkshah1992 <[email protected]> Co-authored-by: Łukasz Magiera <[email protected]> Co-authored-by: zenground0 <[email protected]> Co-authored-by: Steve Loeppky <[email protected]>
Includes:
Related Issues
Proposed Changes
Additional Info
Checklist
Before you mark the PR ready for review, please make sure that: