-
Notifications
You must be signed in to change notification settings - Fork 11
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
Week 06 #43
Week 06 #43
Conversation
I'm trying to use
Shouldn't the redeemer be an integer?
|
Without having looked at the code too much, I imagine the issue is passing a BuiltinInteger vs a PlutusInteger. PlutusInteger is of type PlutusData and the validator always expects PlutusData, never builtin values |
@cffls A proper, deterministic mock context for evaluating UPLC starting from the TxBuilder 👀 |
@@ -0,0 +1,143 @@ | |||
from collections import defaultdict |
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.
Nice! I think it may be a good idea to port this to pycardano.
The mock context should now work as expected (for PlutusV2Scripts!). The property test fails (as expected?). |
Yes, I didn't finish making the property tests correctly yet |
The content for week06 is basically done now. I just need Python-Cardano/pycardano#228 merged and then I can set pycardano to use the main branch dependency. |
Nice work. This advanced the entire ecosystem by a good bit 😄 |
@nielstron it says you requested changes on the pr, could you make an approval (whenever pycardano is ready)? |
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.
Looks great!
fyi for anyone following this, just need to update uplc, pyaiken, pycardano for this to merge |
UPLC and pyaiken are bumped :) |
@nielstron Thanks for the update! We'll use the main branch of pycardano for now until the next release |
No description provided.