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

Memoize TxInfo for every transaction #4851

Open
lehins opened this issue Jan 22, 2025 · 0 comments · May be fixed by #4854
Open

Memoize TxInfo for every transaction #4851

lehins opened this issue Jan 22, 2025 · 0 comments · May be fixed by #4854
Assignees
Labels
💳 technical-debt Issues related to technical debt we introduced performance

Comments

@lehins
Copy link
Collaborator

lehins commented Jan 22, 2025

Every script of the same plutus version shares the same TxInfo, because it does not depend on the redeemer, but on the whole of a transaction.

Currently we are computing the same TxInfo for every script in the transaction, which is redundant computation. This can be mitigated with some memoization that lazily stores TxInfo for every plutus ledger version supported in a particular era.

@lehins lehins added 💳 technical-debt Issues related to technical debt we introduced performance labels Jan 22, 2025
@lehins lehins self-assigned this Jan 22, 2025
@lehins lehins linked a pull request Jan 24, 2025 that will close this issue
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💳 technical-debt Issues related to technical debt we introduced performance
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant