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

Allow client to configure UTxO inclusion in snapshots #808

Merged
merged 6 commits into from
Apr 12, 2023

Conversation

ghost
Copy link

@ghost ghost commented Apr 5, 2023

Why

We had a client request to exclude utxo list in SnapshotConfirmed server outputs

What

In case there is a query param snapshot-utxo with value no remove the utxo field from the output.


  • CHANGELOG updated
  • Documentation updated
  • Added and/or updated haddocks
  • No new TODOs introduced or explained herafter

@github-actions
Copy link

github-actions bot commented Apr 5, 2023

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 2023-04-12 07:11:37.992467528 UTC
Max. memory units 14000000
Max. CPU units 10000000000
Max. tx size (kB) 16384

Script summary

Name Hash Size (Bytes)
νInitial 9492414f8f96e8483a0b8ee268fc06a954641cb2cbaa9a8b093c2c9b 4621
νCommit 5d3f107aaa56d06188cf231941cf8163e777236a9cfdc48fd4bbfa23 2422
νHead 82f16b51e2d81c6f4d42dd7398b4713a445464902f63dfd86ffe754e 8954
μHead 4083fa7081a0f4b4092fb02867c9ac594bb0e8bab8110ab242ba5a72* 4458
  • The minting policy hash is only usable for comparison. As the script is parameterized, the actual script is unique per Head.

Cost of Init Transaction

Parties Tx size % max Mem % max CPU Min fee ₳
1 5052 15.16 6.00 0.54
2 5257 16.09 6.31 0.56
3 5469 20.68 8.11 0.62
5 5874 24.15 9.38 0.67
10 6900 35.74 13.74 0.84
36 12231 99.80 38.02 1.77

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 603 15.75 6.19 0.35

Cost of CollectCom Transaction

Parties UTxO (bytes) Tx size % max Mem % max CPU Min fee ₳
1 57 817 24.50 9.95 0.46
2 114 1138 40.49 16.56 0.65
3 169 1460 59.16 24.34 0.87
4 227 1783 80.24 33.16 1.12

Cost of Close Transaction

Parties Tx size % max Mem % max CPU Min fee ₳
1 640 16.99 7.31 0.37
2 804 18.69 8.19 0.40
3 969 19.87 8.88 0.42
5 1299 22.61 10.40 0.47
10 2125 29.66 14.28 0.59
50 8725 86.05 45.32 1.56

Cost of Contest Transaction

Parties Tx size % max Mem % max CPU Min fee ₳
1 677 21.78 9.12 0.42
2 842 23.92 10.18 0.45
3 1007 25.64 11.06 0.48
5 1336 29.06 12.83 0.54
10 2160 37.67 17.26 0.67
46 8093 99.88 49.25 1.67

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 5174 29.53 12.62 0.71
2 5492 48.00 20.73 0.93
3 5815 69.68 30.31 1.19
4 6138 95.56 41.69 1.49

Cost of FanOut Transaction

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

Parties UTxO UTxO (bytes) Tx size % max Mem % max CPU Min fee ₳
5 0 0 5074 10.59 4.43 0.49
5 1 57 5113 12.12 5.31 0.51
5 5 284 5253 18.22 8.83 0.59
5 10 569 5437 25.85 13.24 0.69
5 20 1141 5799 41.11 22.05 0.90
5 30 1706 6155 56.37 30.86 1.10
5 40 2278 6518 71.64 39.68 1.30
5 50 2851 6876 86.92 48.50 1.50
5 58 3304 7165 99.14 55.56 1.67

@v0d1ch v0d1ch force-pushed the ensemble/client-side-utxo-configuration branch 2 times, most recently from a9fb8a1 to 1683230 Compare April 10, 2023 09:01
@github-actions
Copy link

github-actions bot commented Apr 10, 2023

Test Results

311 tests  +1   305 ✔️ +1   19m 17s ⏱️ -59s
104 suites ±0       6 💤 ±0 
    6 files   ±0       0 ±0 

Results for commit b3c3dbf. ± Comparison against base commit ea67865.

♻️ This comment has been updated with latest results.

@ffakenz ffakenz force-pushed the ensemble/client-side-utxo-configuration branch from 6963ae1 to 617c2a1 Compare April 10, 2023 13:38
@v0d1ch v0d1ch marked this pull request as ready for review April 10, 2023 13:41
@v0d1ch v0d1ch requested review from ch1bo and pgrange April 10, 2023 13:42
@ghost ghost assigned ghost and ffakenz and unassigned ghost Apr 11, 2023
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.

Must: document changes in the api.yaml

@ffakenz ffakenz requested a review from ch1bo April 11, 2023 10:40
@ffakenz ffakenz force-pushed the ensemble/client-side-utxo-configuration branch from c95dea1 to b3c3dbf Compare April 12, 2023 06:57
@v0d1ch v0d1ch merged commit 46abb9d into master Apr 12, 2023
@v0d1ch v0d1ch deleted the ensemble/client-side-utxo-configuration branch April 12, 2023 08:52
@pgrange pgrange added this to the 0.10.0 milestone May 11, 2023
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.

5 participants