feat: f3: update go-f3 to 0.2.0 #12390
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Includes: - fix for excessive bandwidth usage - significant performance improvements - minor consensus fixes Signed-off-by: Jakub Sztandera <oss@kubuxu.com>
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Please update the PR title to match https://github.com/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions
There was a problem hiding this comment.
Please update the PR title to match https://github.com/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions
Signed-off-by: Jakub Sztandera <oss@kubuxu.com>
There was a problem hiding this comment.
Please update the PR title to match https://github.com/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions
Kubuxu
changed the title
There was a problem hiding this comment.
Please update the PR title to match https://github.com/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions
|
🤷 no idea why it is is still angry |
masih
changed the title
github-actions
bot
dismissed
their stale review
PR title now matches the required format.
|
That is different than what is in the doc: https://github.com/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions |
11 tasks
rvagg
changed the title
github-actions
bot
dismissed
their stale review
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://github.com/filecoin-project/lotus/blob/master/CONTRIBUTING.md#pr-title-conventions. Will add |
rjan90
approved these changes
Aug 15, 2024
rjan90
pushed a commit
that referenced
this pull request
Aug 15, 2024
* Update go-f3 to 0.2.0 Includes: - fix for excessive bandwidth usage - significant performance improvements - minor consensus fixes Signed-off-by: Jakub Sztandera <oss@kubuxu.com> * add changelog Signed-off-by: Jakub Sztandera <oss@kubuxu.com> * chore(f3): update to final released version --------- Signed-off-by: Jakub Sztandera <oss@kubuxu.com> Co-authored-by: Steven Allen <steven@stebalien.com>
jennijuju
pushed a commit
that referenced
this pull request
Aug 15, 2024
* Update go-f3 to 0.2.0 Includes: - fix for excessive bandwidth usage - significant performance improvements - minor consensus fixes Signed-off-by: Jakub Sztandera <oss@kubuxu.com> * add changelog Signed-off-by: Jakub Sztandera <oss@kubuxu.com> * chore(f3): update to final released version --------- Signed-off-by: Jakub Sztandera <oss@kubuxu.com> Co-authored-by: Steven Allen <steven@stebalien.com>
rjan90
added a commit
that referenced
this pull request
Aug 19, 2024
…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 <oss@kubuxu.com> * Enable F3 in passive configuration in mainnet config Signed-off-by: Jakub Sztandera <oss@kubuxu.com> * Add changelog Signed-off-by: Jakub Sztandera <oss@kubuxu.com> * add new butterfly assets --------- Signed-off-by: Jakub Sztandera <oss@kubuxu.com> Co-authored-by: Jennifer Wang <jiayingw703@gmail.com> * 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 <steven@stebalien.com> * Update CHANGELOG.md Co-authored-by: Rod Vagg <rod@vagg.org> * Update CHANGELOG.md Co-authored-by: Rod Vagg <rod@vagg.org> * 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 <oss@kubuxu.com> * add changelog Signed-off-by: Jakub Sztandera <oss@kubuxu.com> * chore(f3): update to final released version --------- Signed-off-by: Jakub Sztandera <oss@kubuxu.com> Co-authored-by: Steven Allen <steven@stebalien.com> * 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 <ZenGround0@users.noreply.github.com> * 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 <biglep@filoz.org> * Update CHANGELOG.md * chore: make gen and make docsgen-cli Run `make gen` and `make docsgen-cli` --------- Signed-off-by: Jakub Sztandera <oss@kubuxu.com> Co-authored-by: LexLuthr <88259624+LexLuthr@users.noreply.github.com> Co-authored-by: Jakub Sztandera <oss@kubuxu.com> Co-authored-by: Jennifer Wang <jiayingw703@gmail.com> Co-authored-by: Jiaying Wang <42981373+jennijuju@users.noreply.github.com> Co-authored-by: Steven Allen <steven@stebalien.com> Co-authored-by: Rod Vagg <rod@vagg.org> Co-authored-by: aarshkshah1992 <aarshkshah1992@gmail.com> Co-authored-by: Łukasz Magiera <magik6k@users.noreply.github.com> Co-authored-by: zenground0 <ZenGround0@users.noreply.github.com> Co-authored-by: Steve Loeppky <biglep@filoz.org>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Includes:
Related Issues
Proposed Changes
Additional Info
Checklist
Before you mark the PR ready for review, please make sure that: