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 bdk_coin_select to 0.4 #1559

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

jp1ac4
Copy link
Collaborator

@jp1ac4 jp1ac4 commented Jan 30, 2025

Weights are now stored in the crate as u64 instead of u32 and so we can keep our weights and vsizes as u64.

The tests have been updated to reflect that coin selection now calculates feerates using a transaction's size in rounded-up vbytes, which fixes #1132.

@jp1ac4 jp1ac4 self-assigned this Jan 30, 2025
@jp1ac4 jp1ac4 force-pushed the upgrade-coin-select-crate branch from 26c0da4 to 9cb17a7 Compare January 30, 2025 14:36
Weights are now stored as `u64` instead of `u32`.

The tests have been updated to reflect that coin selection now
calculates feerates using a transaction's size in rounded-up
vbytes.
@jp1ac4 jp1ac4 force-pushed the upgrade-coin-select-crate branch from 9cb17a7 to 1859a2d Compare January 30, 2025 15:46
@jp1ac4
Copy link
Collaborator Author

jp1ac4 commented Jan 30, 2025

Rebased on master.

Copy link
Member

@edouardparis edouardparis left a comment

Choose a reason for hiding this comment

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

ACK 1859a2d

@edouardparis edouardparis merged commit 1d9ff9d into wizardsardine:master Jan 30, 2025
12 checks passed
@jp1ac4 jp1ac4 deleted the upgrade-coin-select-crate branch January 30, 2025 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

spend: feerate using rounded-up vbytes can be lower than target value
2 participants