-
Notifications
You must be signed in to change notification settings - Fork 141
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
implement suggested params #72
Conversation
mjiang102628
commented
Feb 12, 2020
•
edited
Loading
edited
- add SuggestedParams object to handle fee, first valid round, last valid round, and genesis hash
- tests will fail since this is backwards incompatible
- locally updated tests are passing
- add checks for transaction fee exceeding max fee
- switch from cryptography to pycryptodomex; cryptography library missing keccak256 hash function
- use flat_fee before params if fee not specified - use first + 1000 before params if last not specified
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.
Overall, LGTM, this is a very nice incremental improvement.
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.
+1
What about genesisID? Seems like we lack support of it. It can be optional but I don’t think we should skip it. |
genesisID is supported; it's in both the SuggestedParams constructor and the Transaction constructor. |