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 memo plotid #17856
Fix memo plotid #17856
Conversation
…es check for memo and plotid args in create_plots.py
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.
You also need to change the line
plot_memo: bytes32 = stream_plot_info_pk(keys.pool_public_key, keys.farmer_public_key, sk)
and remove the bytes32
plot_memo = stream_plot_info_pk(keys.pool_public_key, keys.farmer_public_key, sk)
I'm also not sure about the bytes or string check - might be better to also fix up the way chia plotters
works instead - although this would work
If we fix earles comment we can get this PR in. |
|
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.
Coverage diff is expected as this didn't have coverage before
Purpose:
Fixes issues where "chia plotters" command is returning bytes and causing type issues. This also removed the bytes32 for the hex arg, which is 128 bytes.
Current Behavior:
New Behavior:
Testing Notes:
Created and duplicated plots using the plot ID and memo using "chia plots create" and "chia plotters chiapos"