-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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: predicate witness data access #1156
Conversation
Although it is working, it is not finished
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.
left some minor remarks
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.
It's looking really good. Just left a minor ask for comments (+ the good suggestions made by everyone else).
close: #1155
Transaction builders do not precompute or estimate predicates anymore when
build()
is called. Instead, this is done before sending the tx to the client. In addition,ScriptTransaction
andCreateTransaction
have a new field:is_using_predicates
to avoid running the estimation code if no predicates are used.Checklist