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

umbrella document for cost model #20517

Closed
wants to merge 1 commit into from

Conversation

tao-stones
Copy link
Contributor

Problem

documentation for cost model is scattered amount PRs and Issues.

Summary of Changes

Add the umbrella doc to

Fixes #

@tao-stones tao-stones requested a review from sakridge October 7, 2021 17:46
Cost is measured in `compute unit`, which describes required computational work (eg., CPU).

Transaction cost is closely echoing proposed Comprehensive Fee Structure (#16984), it is the sum of following components:
1. signature cost, is the number of signatures in a transaction multiplied with fixed signature cost;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jackcmay pointed out that signature cost should not be used in block packing decision since signature verification is done outside slot.

@tao-stones
Copy link
Contributor Author

close it due to dup of #20531

@tao-stones tao-stones closed this Oct 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant