-
Notifications
You must be signed in to change notification settings - Fork 158
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 ex unit evaluation #2380
fix ex unit evaluation #2380
Conversation
640711e
to
182f5b7
Compare
182f5b7
to
13157f7
Compare
- round trip test for translating exunits to/from plutus - test whether evaluateTransactionExecutionUnits provides enough exUnits - fixed ex unit translation
13157f7
to
e07983b
Compare
41cad68
to
d66edb5
Compare
scr | ||
datums | ||
evalWithTightBudget :: ShortByteString -> [P.Data] -> Either P.EvaluationError () | ||
evalWithTightBudget scr datums = do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
much better, thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for making the tests better and adding new tests!
base16-bytestring, | ||
bytestring, | ||
cardano-binary, | ||
cardano-ledger-alonzo, | ||
cardano-ledger-alonzo-test, | ||
cardano-ledger-shelley-ma, | ||
cardano-ledger-core, | ||
cardano-ledger-test, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is going to be a bit confusing, since I think that package will ultimately want to depend on this one
provides enough exUnits