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

Rework plutus-ledger-api tests for new cost model machinery #6089

Open
kwxm opened this issue May 22, 2024 · 0 comments
Open

Rework plutus-ledger-api tests for new cost model machinery #6089

kwxm opened this issue May 22, 2024 · 0 comments
Labels
Costing Anything relating to costs, fees, gas, etc. Internal Plutus Ledger API status: triaged

Comments

@kwxm
Copy link
Contributor

kwxm commented May 22, 2024

PR #6087 introduces multiple cost models for Plutus Core and increases the configurability of cost models. plutus-ledger-api contains many tests for the cost model parameters which are now not very meaningful (for example, they used to check that the V1 cost model parameters are a subset of the V2 parameters, and that those are a subset of the V3 parameters, which is no longer true). These tests have been adjusted so that they pass for now, but the whole thing really needs to be completely reworked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Costing Anything relating to costs, fees, gas, etc. Internal Plutus Ledger API status: triaged
Projects
None yet
Development

No branches or pull requests

2 participants