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.

Sign up for GitHub

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

Jump to bottom

checkpoint: into main from release/2.1.4 @ 55c064a29215d78393b3b37f2c3029c100560e12 #17141

Merged
merged 3 commits into from
Dec 22, 2023
Merged

checkpoint: into main from release/2.1.4 @ 55c064a29215d78393b3b37f2c3029c100560e12 #17141

merged 3 commits into from
Dec 22, 2023

Conversation

altendky
Copy link
Contributor

Source hash: 55c064a
Remaining commits: 0

arvidn and others added 2 commits December 21, 2023 18:02
@arvidn
improve performance of total_mempool_fees() and `total_mempool_cost…
55c064a 
…()` (#17107)

* improve performance of total_mempool_fees() and total_mempool_cost()

* log when updating the mempool using the slow-path

* extend test to ensure total-cost and total-fee book-keeping stays in sync with the DB

* lower mempool size from 50 blocks to 10 blocks
@altendky
Merge commit '55c064a29215d78393b3b37f2c3029c100560e12' into checkpoi…
1df7f5f 
…nt/main_from_release_2.1.4_55c064a29215d78393b3b37f2c3029c100560e12
@altendky altendky requested a review from a team as a code owner December 22, 2023 01:42
@altendky altendky added checkpoint_merge Changed Required label for PR that categorizes merge commit message as "Changed" for changelog labels Dec 22, 2023
@coveralls-official Coveralls Official
Copy link

coveralls-official bot commented Dec 22, 2023 •
edited
Loading

Pull Request Test Coverage Report for Build 7298444617

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

  • 65 of 65 (100.0%) changed or added relevant lines in 5 files are covered.
  • 171 unchanged lines in 17 files lost coverage.
  • Overall coverage decreased (-0.1%) to 90.452%
Files with Coverage Reduction New Missed Lines %
chia/server/address_manager.py 1 90.19%
chia/wallet/util/wallet_sync_utils.py 1 77.67%
tests/wallet/cat_wallet/test_cat_wallet.py 1 99.85%
chia/data_layer/data_layer.py 2 81.26%
chia/data_layer/data_layer_wallet.py 2 90.98%
chia/timelord/timelord_api.py 2 87.85%
chia/farmer/farmer_api.py 3 92.73%
chia/wallet/did_wallet/did_wallet_puzzles.py 3 85.9%
tests/wallet/nft_wallet/test_nft_puzzles.py 3 97.39%
chia/server/server.py 4 79.66%
Totals Coverage Status
Change from base Build 7292945368: -0.1%
Covered Lines: 93928
Relevant Lines: 103797
💛 - Coveralls

arvidn
arvidn reviewed Dec 22, 2023
View reviewed changes
arvidn
arvidn previously approved these changes Dec 22, 2023
View reviewed changes
@cmmarslender cmmarslender merged commit 739b0d8 into main Dec 22, 2023
@cmmarslender cmmarslender deleted the checkpoint/main_from_release_2.1.4_55c064a29215d78393b3b37f2c3029c100560e12 branch December 22, 2023 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AmineKhaldi AmineKhaldi AmineKhaldi approved these changes

@arvidn arvidn arvidn approved these changes

@emlowe emlowe Awaiting requested review from emlowe

Assignees
No one assigned
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog checkpoint_merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@altendky @AmineKhaldi @arvidn @cmmarslender