Skip to content
This repository was archived by the owner on Dec 18, 2024. It is now read-only.

Use FeePolynomial in WeightToFee implementation. #41

Merged
stiiifff merged 1 commit intomainfrom
wzy-fee-polynomial
Jul 14, 2023
Merged

Use FeePolynomial in WeightToFee implementation. #41
stiiifff merged 1 commit intomainfrom
wzy-fee-polynomial

Conversation

@weezy20
Copy link
Copy Markdown
Contributor

@weezy20 weezy20 commented Jun 30, 2023 •
edited
Loading

Closes #37

@weezy20 weezy20 changed the title Use FeePolynomial in WeightToFee implementation. Close #37 Use FeePolynomial in WeightToFee implementation. Jun 30, 2023
@weezy20 weezy20 requested a review from stiiifff June 30, 2023 11:19
@weezy20
Copy link
Copy Markdown
Contributor Author

weezy20 commented Jul 9, 2023

I think we should hold out on merging this one before #44 is merged in order to avoid another rebase.

@stiiifff
Copy link
Copy Markdown
Contributor

stiiifff commented Jul 14, 2023 •
edited
Loading

@weezy20 LGTM (this is a small one, it shouldn't be blocked by #44 which is much bigger and will take more time to review).

@stiiifff stiiifff merged commit 313f18d into main Jul 14, 2023
@stiiifff stiiifff deleted the wzy-fee-polynomial branch July 14, 2023 09:02
@weezy20 weezy20 mentioned this pull request Jul 14, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@stiiifff stiiifff stiiifff approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Use FeePolynomial instead of WeightCoefficientCalc

2 participants

@weezy20 @stiiifff