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

NFT example fails #72

Closed
samuelWilliams99 opened this issue Mar 7, 2022 · 6 comments · Fixed by #74
Closed

NFT example fails #72

samuelWilliams99 opened this issue Mar 7, 2022 · 6 comments · Fixed by #74
Labels
bug Something isn't working

Comments

@samuelWilliams99
Copy link
Contributor

"The budget was overspent" error on testnet
@samuelWilliams99 samuelWilliams99 added the bug Something isn't working label Mar 7, 2022
@szg251
Copy link
Collaborator

szg251 commented Mar 10, 2022

We just upgraded the plutus version, so that could be the culprit..

@samuelWilliams99
Copy link
Contributor Author

It's also possibly a result of my balancing changes - perhaps an edge case that tests didn't consider?

@samuelWilliams99
Copy link
Contributor Author

Depends what budget means here - is it token budget or computation/memory budget?

@mikekeke
Copy link
Collaborator

Plutip currently for script like this: https://github.com/mlabs-haskell/plutip/blob/bug/budget-overspent/test/Spec/SpendingContract.hs#L20-L21
which used in contract here: https://github.com/mlabs-haskell/plutip/blob/bug/budget-overspent/test/Spec/SpendingContract.hs#L70-L72

throws this:

CekError An error has occurred:  User error:\\\\nThe budget was overspent. Final negative state: ({ cpu: -63140 | mem: 4003 })

@samuelWilliams99
Copy link
Contributor Author

ah, so cpu budget estimation is pretty far off

@samuelWilliams99 samuelWilliams99 mentioned this issue Mar 10, 2022
Closed
@samuelWilliams99
Copy link
Contributor Author

Found out what it is, created the above issue :)

@samuelWilliams99 samuelWilliams99 linked a pull request Mar 11, 2022 that will close this issue
Add Script context to budget estimation #74
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Script context to budget estimation mlabs-haskell/bot-plutus-interface
3 participants
@mikekeke @szg251 @samuelWilliams99