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

Factor CardanoClient queries #360

Merged
merged 1 commit into from
May 24, 2022
Merged

Conversation

ch1bo
Copy link
Member

@ch1bo ch1bo commented May 23, 2022

This allows us to drop the hydra-cluster dependency in hydra-tui, which I used on the babbage-preview branch (where hydra-cluster is not yet updated, but hydra-tui works already)

@ch1bo ch1bo requested review from KtorZ and a user May 23, 2022 13:28
@github-actions
Copy link

github-actions bot commented May 23, 2022

Unit Test Results

    5 files  ±0    83 suites  ±0   8m 1s ⏱️ +19s
221 tests ±0  219 ✔️ ±0  2 💤 ±0  0 ±0 

Results for commit 7d12459. ± Comparison against base commit 95be983.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented May 23, 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-05-24 09:08:12.862500249 UTC
Max. memory units 14000000.00
Max. CPU units 10000000000.00
Max. tx size (kB) 16384

Cost of Init Transaction

Parties Tx size % max Mem % max CPU
1 4737 27.12 15.80
2 4937 29.19 16.85
3 5138 29.97 17.12
5 5536 16.82 8.67
10 6537 33.86 18.01
30 10538 66.91 33.98
43 13140 95.01 48.39

Cost of Commit Transaction

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

UTxO Tx size % max Mem % max CPU
1 5860 17.98 8.81

Cost of CollectCom Transaction

Parties Tx size % max Mem % max CPU
1 13517 19.77 10.61
2 14150 39.12 21.39
3 14498 58.41 32.17
4 14548 76.07 41.96

Cost of Close Transaction

Parties Tx size % max Mem % max CPU
1 9264 8.16 4.33
2 9656 11.86 6.32
3 9824 12.56 6.70
5 10369 17.88 9.53
10 11350 24.18 12.95
30 15286 59.74 32.12
55 16265 70.98 37.40

Cost of Abort Transaction

Parties Tx size % max Mem % max CPU

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
1 13369 13.49 7.91
2 13271 10.17 5.87
3 13368 13.10 7.57
5 13439 15.87 9.15
10 13607 23.89 13.77
50 15019 85.71 49.34
60 15223 97.47 56.09

-- XXX(SN): this is a hydra implementation detail and should be moved
-- somewhere hydra specific
let utxoWithoutFuel = Map.filter (not . isMarkedOutput) (UTxO.toMap utxo)
continue $ s & dialogStateL .~ commitDialog utxoWithoutFuel
_ ->
continue $ s & feedbackL ?~ UserFeedback Error "Invalid command."
where
ourAddress =
Copy link

Choose a reason for hiding this comment

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

👍 ♡ the name

@ch1bo ch1bo force-pushed the ch1bo/remove-hydra-cluster-dep-in-tui branch 2 times, most recently from 25eb2f6 to 7d12459 Compare May 24, 2022 08:58
@ch1bo ch1bo merged commit d381775 into master May 24, 2022
@ch1bo ch1bo deleted the ch1bo/remove-hydra-cluster-dep-in-tui branch May 24, 2022 17:45
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.

1 participant