Skip to content
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

bump farmer fill rate to 70% #17954

Merged
merged 2 commits into from
May 7, 2024
Merged

bump farmer fill rate to 70% #17954

merged 2 commits into from
May 7, 2024

Conversation

arvidn
Copy link
Contributor

@arvidn arvidn commented May 2, 2024

Purpose:

Have farmers create slightly larger blocks

Current Behavior:

Blocks are filled to 60% of capacity

New Behavior:

Blocks are filled to 70% of capacity

@arvidn arvidn requested a review from a team as a code owner May 2, 2024 16:44
@arvidn arvidn requested a review from wjblanke May 2, 2024 16:44
@arvidn arvidn added the Changed Required label for PR that categorizes merge commit message as "Changed" for changelog label May 2, 2024
wjblanke
wjblanke previously approved these changes May 2, 2024
Copy link
Contributor

@wjblanke wjblanke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aok

Adapt test_create_bundle_from_mempool_on_max_cost.
@arvidn arvidn added the ready_to_merge Submitter and reviewers think this is ready label May 7, 2024
@pmaslana pmaslana merged commit 1dc157e into main May 7, 2024
358 checks passed
@pmaslana pmaslana deleted the bump-fill-rate-0.7 branch May 7, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog ready_to_merge Submitter and reviewers think this is ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants