Skip to content
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

Add tools executable to generate Hydra key pair #474

Merged
4 commits merged into from
Sep 5, 2022

Conversation

ghost
Copy link

@ghost ghost commented Aug 30, 2022

This is related to #189: Trying to spin-up a node myself I realised there wasn't any easy way to generate those keys so I wrote this small tool.

@ghost ghost requested review from ch1bo and ffakenz August 30, 2022 15:41
@ghost ghost self-assigned this Aug 30, 2022
@github-actions
Copy link

github-actions bot commented Aug 30, 2022

Transactions Costs

Sizes and execution budgets for Hydra protocol transactions. Note that unlisted parameters are currently using arbitrary values and results are not fully deterministic and comparable to previous runs.

Metadata
Generated at 2022-09-03 17:22:36.261317866 UTC
Max. memory units 14000000
Max. CPU units 10000000000
Max. tx size (kB) 16384

Cost of Init Transaction

Parties Tx size % max Mem % max CPU Min fee ₳
1 4832 10.43 4.13 0.48
2 5036 12.80 5.06 0.51
3 5242 14.34 5.64 0.54
5 5653 16.90 6.58 0.58
10 6677 27.31 10.61 0.74
46 14059 98.64 38.10 1.84

Cost of Commit Transaction

Currently only one UTxO per commit allowed (this is about to change soon)

UTxO Tx size % max Mem % max CPU Min fee ₳
1 5768 19.83 8.00 0.62

Cost of CollectCom Transaction

Parties Tx size % max Mem % max CPU Min fee ₳
1 13066 20.77 8.32 0.95
2 13379 36.50 14.78 1.14
3 13701 54.93 22.42 1.36
4 14094 77.44 31.78 1.63

Cost of Close Transaction

Parties Tx size % max Mem % max CPU Min fee ₳
1 9329 8.19 3.36 0.65
2 9559 9.80 4.14 0.68
3 9723 10.63 4.61 0.70
5 10021 11.70 5.32 0.72
10 10879 16.04 7.75 0.81
30 14216 32.26 17.02 1.16
69 16107 41.03 15.53 1.30

Cost of Contest Transaction

Parties Tx size % max Mem % max CPU Min fee ₳
1 9365 8.56 3.49 0.66
2 9518 9.38 3.96 0.67
3 9693 10.16 4.42 0.69
5 10022 11.68 5.30 0.72
10 10883 16.25 7.82 0.82
30 14189 32.21 16.99 1.16
43 16368 42.74 23.01 1.38

Cost of Abort Transaction

Some variation because of random mixture of still initial and already committed outputs.

Parties Tx size % max Mem % max CPU Min fee ₳
1 13703 25.73 10.91 1.04
2 13605 34.02 14.28 1.13
3 13715 47.64 20.16 1.28
4 14397 79.24 34.79 1.67

Cost of FanOut Transaction

Involves spending head output and burning head tokens. Uses ada-only UTxO for better comparability.

UTxO Tx size % max Mem % max CPU Min fee ₳
1 13484 10.26 4.46 0.86
2 13584 12.16 5.49 0.89
3 13556 13.55 6.32 0.90
5 13627 16.64 8.10 0.94
10 13744 23.87 12.35 1.04
50 15246 84.88 47.60 1.85
59 15639 98.98 55.68 2.04

Copy link
Member

@ch1bo ch1bo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Slight YAGNI: What if we never need other things than keygen? Naming the executable what you have as subcommand would be more on point, for now.

Copy link
Member

@ch1bo ch1bo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Forgot something:

@ghost
Copy link
Author

ghost commented Aug 31, 2022

Yeah, I agree with the YAGNI :) What about moving the publish-scripts command there (later, in another PR)?

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
This is related to #189: Trying to spin-up a node myself I realised
there wasn't any easy way to generate those keys so I wrote this small
tool.
@ghost ghost force-pushed the abailly-iohk/generate-hydra-key branch from 7865e9e to d5c8bb5 Compare September 1, 2022 16:42
@ghost
Copy link
Author

ghost commented Sep 1, 2022

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@ghost ghost requested a review from ch1bo September 2, 2022 06:51
@ghost
Copy link
Author

ghost commented Sep 2, 2022

I have added another tiny command to output the marker datum hash, something I have been needing while experimenting with deploying hydraw.

Copy link
Member

@ch1bo ch1bo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still missing:

@ghost ghost force-pushed the abailly-iohk/generate-hydra-key branch from 8a044b3 to d5c8bb5 Compare September 2, 2022 08:11
@github-actions
Copy link

github-actions bot commented Sep 2, 2022

Unit Test Results

247 tests  ±0   241 ✔️ ±0   14m 4s ⏱️ -7s
  89 suites ±0       6 💤 ±0 
    5 files   ±0       0 ±0 

Results for commit b8fbf41. ± Comparison against base commit ac49018.

♻️ This comment has been updated with latest results.

@ghost
Copy link
Author

ghost commented Sep 3, 2022

@ch1bo I forgot, thanks for the reminder

@ghost ghost requested a review from ch1bo September 3, 2022 10:19
@ghost ghost merged commit 5845758 into master Sep 5, 2022
@ghost ghost deleted the abailly-iohk/generate-hydra-key branch September 5, 2022 09:41
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants