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

Fix data_receipt_creation_per_byte in runtime-params-estimator #12781

Open
jancionear opened this issue Jan 23, 2025 · 0 comments
Open

Fix data_receipt_creation_per_byte in runtime-params-estimator #12781

jancionear opened this issue Jan 23, 2025 · 0 comments
Labels
A-contract-runtime Area: contract compilation and execution, virtual machines, etc

Comments

@jancionear
Copy link
Contributor

jancionear commented Jan 23, 2025

data_receipt_creation_per_byte used to generate a thousand 10 byte receipts and then another thousand of 100kB ones.
It expects all receipts to be processed within 2 blocks, but 1000 * 100kB = 100MB and bandwidth scheduler allows to send only 4.5MB per height, so with the scheduler enabled it takes ~25 blocks to send 100MB of receipts. Because of that the number of receipts was reduced from 1000 to 40. Now all of them are processed within the expected two blocks, but 40 receipts are processed too quickly to properly estimate the cost.

data_receipt_creation_per_byte should probably be rewritten to better estimate the cost of creating receipts.

Refs zulip conversation

@jancionear jancionear added the A-contract-runtime Area: contract compilation and execution, virtual machines, etc label Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-contract-runtime Area: contract compilation and execution, virtual machines, etc
Projects
None yet
Development

No branches or pull requests

1 participant