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.

Sign up for GitHub

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

Jump to bottom

Update MissingScript error in evaluateTransactionExecutionUnits #3593

Merged
merged 1 commit into from Feb 17, 2022
Merged

Update MissingScript error in evaluateTransactionExecutionUnits #3593

merged 1 commit into from Feb 17, 2022

Conversation

Jimbo4350
Copy link
Contributor

No description provided.

@Jimbo4350 Jimbo4350 force-pushed the jordan/fix-build-impossible-error branch from 3668481 to 5f33c84 Compare February 10, 2022 17:16
JaredCorduan
JaredCorduan approved these changes Feb 10, 2022
View reviewed changes
Copy link
Contributor

@JaredCorduan JaredCorduan left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@Jimbo4350
Copy link
Contributor Author

bors r+

iohk-bors bot added a commit that referenced this pull request Feb 17, 2022
@iohk-bors @Jimbo4350
Merge #3589 #3593
e3440b6 
3589: Update  cli build command to calculate Plutus script cost r=Jimbo4350 a=Jimbo4350

Resolves: #3590

 We introduce the  following change:
```
cardano-cli transaction build 
...
(--out-file FILE | --calculate-plutus-script-cost FILE)
```
If you are only interested in the script cost, specify the `--calculate-plutus-script-cost` option.

Example: 

```
$CARDANO_CLI transaction build \
  --alonzo-era \
  --cardano-mode \
  --testnet-magic "$TESTNET_MAGIC" \
  --change-address "$utxoaddr" \
  --tx-in "$plutusutxotxin" \
  --tx-in-collateral "$txinCollateral" \
  --tx-out "$dummyaddress+10000000" \
  --tx-in-script-file "$plutusscriptinuse" \
  --tx-in-datum-file "$datumfilepath"  \
  --protocol-params-file "$WORK/pparams.json" \
  --tx-in-redeemer-file "$redeemerfilepath" \
  --calculate-plutus-script-cost "$WORK/create-datum-output.scriptcost"
  
  > cat $WORK/create-datum-output.scriptcost
  [
    {
        "executionUnits": {
            "memory": 1700,
            "steps": 476468
        },
        "lovelaceCost": 133,
        "scriptHash": "67f33146617a5e61936081db3b2117cbf59bd2123748f58ac9678656"
    }
]
  
  ```

3593: Update MissingScript error in evaluateTransactionExecutionUnits r=Jimbo4350 a=Jimbo4350



Co-authored-by: Jordan Millar <jordan.millar@iohk.io>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Feb 17, 2022

This PR was included in a batch that successfully built, but then failed to merge into master (it was a non-fast-forward update). It will be automatically retried.

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Feb 17, 2022

Build succeeded:

@iohk-bors iohk-bors bot merged commit 600f5d2 into master Feb 17, 2022
@iohk-bors iohk-bors bot deleted the jordan/fix-build-impossible-error branch February 17, 2022 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JaredCorduan JaredCorduan JaredCorduan approved these changes

@DavidEichmann DavidEichmann Awaiting requested review from DavidEichmann

@dcoutts dcoutts Awaiting requested review from dcoutts

@erikd erikd Awaiting requested review from erikd

@newhoggy newhoggy Awaiting requested review from newhoggy

@nc6 nc6 Awaiting requested review from nc6

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Jimbo4350 @JaredCorduan