-
Notifications
You must be signed in to change notification settings - Fork 483
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
Plutus Core spec for Batch 4 inconsistent with conformance tests #6761
Labels
Comments
kwxm
added
status: triaged
and removed
status: needs triage
GH issues that requires triage
labels
Dec 23, 2024
Thanks for pointing those out. You're correct about both of them. The width check is done here and indeed it only fails if the length is strictly greater than 8192. We'll fix these shortly. |
Merged
I think these should be fixed by #6762. |
I'm closing the issue then. Please reopen if something's still off. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Summary
The plutus core spec of Batch 4, Note 1 contains: "An integer width argument π€ with 0 β€ π€ < 8192"
The conformance test /builtin/semantics/integerToByteString/big-endian/bounded/max-input-fits-max-width contradicts this by providing width 8192 explicitly.
Also the spec for ByteStringToInteger is [ππππ, ππ’ππππππππ] β ππ’ππππππππ but should be [ππππ, ππ’ππππππππ] β integer
The text was updated successfully, but these errors were encountered: