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

499 Add note about ipv6 addresses in the docs #500

Merged
merged 4 commits into from
Sep 21, 2022

Conversation

v0d1ch
Copy link
Contributor

@v0d1ch v0d1ch commented Sep 19, 2022

fix #499

@v0d1ch v0d1ch requested review from pgrange, ch1bo, ffakenz and a user September 19, 2022 16:45
@v0d1ch v0d1ch self-assigned this Sep 19, 2022
@github-actions
Copy link

Unit Test Results

253 tests  ±0   247 ✔️ ±0   14m 2s ⏱️ - 2m 2s
  91 suites ±0       6 💤 ±0 
    5 files   ±0       0 ±0 

Results for commit 6bdaa2d. ± Comparison against base commit b0ebec9.

@github-actions
Copy link

github-actions bot commented Sep 19, 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-20 11:22:48.763338071 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 9.48 3.74 0.47
2 5037 12.01 4.73 0.50
3 5241 16.37 6.48 0.56
5 5653 19.13 7.50 0.61
10 6678 27.97 10.88 0.75
45 13858 98.50 38.09 1.83

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 5771 19.93 8.06 0.62

Cost of CollectCom Transaction

Parties Tx size % max Mem % max CPU Min fee ₳
1 13066 20.60 8.25 0.95
2 13380 36.46 14.77 1.14
3 13809 56.94 23.24 1.39
4 14061 77.20 31.67 1.62

Cost of Close Transaction

Parties Tx size % max Mem % max CPU Min fee ₳
1 9323 8.19 3.36 0.65
2 9530 9.41 3.98 0.67
3 9653 9.75 4.27 0.68
5 10020 11.75 5.34 0.72
10 10879 16.09 7.77 0.81
30 14216 32.26 17.02 1.16
67 15910 39.93 15.12 1.28

Cost of Contest Transaction

Parties Tx size % max Mem % max CPU Min fee ₳
1 9363 8.56 3.49 0.66
2 9495 8.94 3.79 0.67
3 9697 10.35 4.49 0.69
5 10056 12.12 5.47 0.73
10 10849 15.62 7.57 0.81
30 14223 32.42 17.07 1.16
43 16364 42.55 22.94 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 13494 22.50 9.35 0.99
2 14101 44.30 19.59 1.27
3 13715 47.64 20.16 1.28
4 14321 77.57 33.93 1.65

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 13550 10.89 4.71 0.87
2 13456 11.40 5.19 0.87
3 13623 13.68 6.37 0.91
5 13563 16.21 7.93 0.94
10 13804 24.25 12.50 1.04
50 15243 84.88 47.60 1.85
59 15504 98.22 55.38 2.03

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Minor suggestion

docs/docs/getting-started/demo/without-docker.md Outdated Show resolved Hide resolved
@v0d1ch v0d1ch force-pushed the 499-add-note-about-ipv6-addresses-in-the-docs branch from 6bdaa2d to 7d04783 Compare September 20, 2022 06:57
@v0d1ch v0d1ch merged commit 60cce2f into master Sep 21, 2022
@v0d1ch v0d1ch deleted the 499-add-note-about-ipv6-addresses-in-the-docs branch September 21, 2022 12:21
@ch1bo
Copy link
Member

ch1bo commented Sep 21, 2022

@v0d1ch Note that this was not passing CI when you merged it because of the flaky test fixed in #493. We usually would try to have a green CI and also make PRs fast-forward before merging. These are obviously unrelated changes so this is fine, but I though you should know going forward :)

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.

Add note about ipv6 addresses in the docs
3 participants