Skip to content

Conversation

Unisay
Copy link
Contributor

@Unisay Unisay commented Jul 22, 2025

add version info and clarify dependencis for CompiledCode evaluation

@Unisay Unisay requested a review from zliu41 July 22, 2025 15:08
@Unisay Unisay self-assigned this Jul 22, 2025
Copy link
Contributor

github-actions bot commented Jul 22, 2025

PR Preview Action v1.6.2

🚀 View preview at
https://IntersectMBO.github.io/plutus/pr-preview/docs/pr-7230/

Built to branch gh-pages at 2025-09-22 13:29 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@Unisay Unisay added the No Changelog Required Add this to skip the Changelog Check label Jul 24, 2025
@Unisay Unisay requested a review from ana-pantilie September 12, 2025 17:28
build-depends:
plutus-tx,
plutus-tx:plutus-tx-testlib,
plutus-ledger-api
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do you need to add plutus-ledger-api? Wouldn't plutus-tx be enough?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

PlutusLedgerAPI provides realistic machine parameters (contains cost model) to the evaluator:

import PlutusLedgerApi.Common (PlutusLedgerLanguage (..), alonzoPV)
import PlutusLedgerApi.MachineParameters (machineParametersFor)

...

result = evaluateCompiledCode'
    -- requires import PlutusLedgerApi.MachineParameters:
    (machineParametersFor PlutusV2 alonzoPV)
    (compiledCode `unsafeApplyCode` argumentCompiled)

@Unisay Unisay enabled auto-merge (squash) September 22, 2025 13:27
@Unisay Unisay merged commit a2407e0 into master Sep 23, 2025
7 checks passed
@Unisay Unisay deleted the yura/improve-doc branch September 23, 2025 02:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No Changelog Required Add this to skip the Changelog Check
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants