-
Notifications
You must be signed in to change notification settings - Fork 87
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
Update documentation related to running on mainnet #795
Conversation
Transactions CostsSizes and execution budgets for Hydra protocol transactions. Note that unlisted parameters are currently using
Script summary
Cost of Init Transaction
Cost of Commit TransactionCurrently only one UTxO per commit allowed (this is about to change soon)
Cost of CollectCom Transaction
Cost of Close Transaction
Cost of Contest Transaction
Cost of Abort TransactionSome variation because of random mixture of still initial and already committed outputs.
Cost of FanOut TransactionInvolves spending head output and burning head tokens. Uses ada-only UTxO for better comparability.
|
Test Results297 tests - 11 291 ✔️ - 11 21m 36s ⏱️ -36s Results for commit cd56f52. ± Comparison against base commit 2cf8aba. This pull request removes 11 tests.
♻️ This comment has been updated with latest results. |
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.
This review tries to apply the perfection game protocol
I will describe what I like about this P.R. and then what improvement I can think of to make it perfect.
Ask for clarifications if my comments are not clear.
If you disagree with my comments, just ignore them and don’t answer or explain.
Hence I, in advance, approve the P.R.
For me to find it perfect you would have to:
- be explicit in Mainnet compatibility #713 that we still need to put the appropriate warnings in the appropriate places
- see other improvement proposals inline
33c9a67
to
bbb4b8e
Compare
Co-authored-by: Pascal Grange <pgrange@users.noreply.github.com>
Co-authored-by: Pascal Grange <pgrange@users.noreply.github.com>
Co-authored-by: Pascal Grange <pgrange@users.noreply.github.com>
|
||
> Note: This setup is meant to configure your cluster for devnet network. If you want to run the node on mainnet check out _Running on Mainnet_ paragraph. | ||
|
||
## Running on Mainnet |
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.
@v0d1ch What about running on preview/preprod testnets before? I think we should suggest that at and IMO this section would be better if it explains more generally how the hydra-node needs to be configured for a given network. Maybe even a bit earlier in this document (it feels out of place here).
|
||
+ If you run into this, file an issue and the Hydra team will be able to help | ||
you finalize your head in a manual workaround. |
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.
@v0d1ch Why was this point about reference scripts removed? This is still relevant!
Small changes to the documentation related to running on the mainnet.