Skip to content

fix(shed): account for new sectors that haven't been proven yet#13042

Merged
rvagg merged 1 commit intomasterfrom
rvagg/miner-fees-new-sectors
Apr 18, 2025
Merged

fix(shed): account for new sectors that haven't been proven yet#13042
rvagg merged 1 commit intomasterfrom
rvagg/miner-fees-new-sectors

Conversation

@rvagg
Copy link
Copy Markdown
Member

@rvagg rvagg commented Apr 17, 2025

original code assumed a static state, but that's just not how mainnet works; we need to account for constantly onboarding sectors and ensure we're not counting fees for sectors that we don't expect to have been paid yet because they won't have been proven

@rvagg rvagg added the skip/changelog This change does not require CHANGELOG.md update label Apr 17, 2025
@rvagg rvagg requested a review from rjan90 April 17, 2025 00:51
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FilOz Apr 17, 2025
original code assumed a static state, but that's just not how mainnet works; we
need to account for constantly onboarding sectors and ensure we're not counting
fees for sectors that we don't expect to have been paid yet because they
won't have been proven
@rvagg rvagg force-pushed the rvagg/miner-fees-new-sectors branch from 4609cef to 6ac7f5c Compare April 17, 2025 00:59
@github-project-automation github-project-automation Bot moved this from 📌 Triage to ✔️ Approved by reviewer in FilOz Apr 17, 2025
@rvagg rvagg merged commit 54ec502 into master Apr 18, 2025
91 checks passed
@rvagg rvagg deleted the rvagg/miner-fees-new-sectors branch April 18, 2025 09:45
@github-project-automation github-project-automation Bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz Apr 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip/changelog This change does not require CHANGELOG.md update

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

2 participants