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.

Sign up for GitHub

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

Jump to bottom

fix(ReadParams): return protocol parameter values #287

Merged
merged 1 commit into from
Nov 7, 2024
Merged

fix(ReadParams): return protocol parameter values #287

merged 1 commit into from
Nov 7, 2024

Conversation

verbotenj
Copy link
Contributor

Server response:

CCardano: cardano:{coins_per_utxo_byte:4310  max_tx_size:16384  min_fee_coefficient:44  min_fee_constant:155381  max_block_body_size:90112  max_block_header_size:1100  stake_key_deposit:2000000  pool_deposit:500000000  pool_retirement_epoch_bound:18  desired_number_of_pools:500  pool_influence:{numerator:3  denominator:10}  monetary_expansion:{numerator:3  denominator:1000}  treasury_expansion:{numerator:1  denominator:5}  min_pool_cost:170000000  protocol_version:{major:9}  max_value_size:5000  collateral_percentage:150  max_collateral_inputs:3  cost_models:{plutus_v1:{values:100788  values:420  values:1  values:1  values:1000  values:173  values:0  values:1  values:1000  values:59957  values:4  values:1  values:11183  values:32  values:201305  values:8356  values:4  values:16000  values:100  values:16000  values:100  values:16000  values:100  values:16000  values:100  values:16000  values:100  values:16000  values:100  values:100  values:100  values:16000  values:100  values:94375  values:32  values:132994  values:32  values:61462  values:4  values:72010  values:178  values:0  values:1  values:22151  values:32  values:91189  values:769  values:4  values:2  values:85848  values:228465  values:122  values:0  values:1  values:1  values:1000  values:42921  values:4  values:2  values:24548  values:29498  values:38  values:1  values:898148  values:27279  values:1  values:51775  values:558  values:1  values:39184  values:1000  values:60594  values:1  values:141895  values:32  values:83150  values:32  values:15299  values:32  values:76049  values:1  values:13169  values:4  values:22100  values:10  values:28999  values:74  values:1  values:28999  values:74  values:1  values:43285  values:552  values:1  values:44749  values:541  values:1  values:33852  values:32  values:68246  values:32  values:72362  values:32  values:7243  values:32  values:7391  values:32  values:11546  values:32  values:85848  values:228465  values:122  values:0  values:1  values:1  values:90434  values:519  values:0  values:1  values:74433  values:32  values:85848  values:228465  values:122  values:0  values:1  values:1  values:85848  values:228465  values:122  values:0  values:1  values:1  values:955506  values:213312  values:0  values:2  values:270652  values:22588  values:4  values:1457325  values:64566  values:4  values:20467  values:1  values:4  values:0  values:141992  values:32  values:100788  values:420  values:1  values:1  values:81663  values:32  values:59498  values:32  values:20142  values:32  values:24588  values:32  values:20744  values:32  values:25933  values:32  values:24623  values:32  values:43053543  values:10  values:53384111  values:14333  values:10  values:43574283  values:26308  values:10}  plutus_v2:{values:100788  values:420  values:1  values:1  values:1000  values:173  values:0  values:1  values:1000  values:59957  values:4  values:1  values:11183  values:32  values:201305  values:8356  values:4  values:16000  values:100  values:16000  values:100  values:16000  values:100  values:16000  values:100  values:16000  values:100  values:16000  values:100  values:100  values:100  values:16000  values:100  values:94375  values:32  values:132994  values:32  values:61462  values:4  values:72010  values:178  values:0  values:1  values:22151  values:32  values:91189  values:769  values:4  values:2  values:85848  values:123203  values:7305  values:-900  values:1716  values:549  values:57  values:85848  values:0  values:1  values:1  values:1000  values:42921  values:4  values:2  values:24548  values:29498  values:38  values:1  values:898148  values:27279  values:1  values:51775  values:558  values:1  values:39184  values:1000  values:60594  values:1  values:141895  values:32  values:83150  values:32  values:15299  values:32  values:76049  values:1  values:13169  values:4  values:22100  values:10  values:28999  values:74  values:1  values:28999  values:74  values:1  values:43285  values:552  values:1  values:44749  values:541  values:1  values:33852  values:32  values:68246  values:32  values:72362  values:32  values:7243  values:32  values:7391  values:32  values:11546  values:32  values:85848  values:123203  values:7305  values:-900  values:1716  values:549  values:57  values:85848  values:0  values:1  values:90434  values:519  values:0  values:1  values:74433  values:32  values:85848  values:123203  values:7305  values:-900  values:1716  values:549  values:57  values:85848  values:0  values:1  values:1  values:85848  values:123203  values:7305  values:-900  values:1716  values:549  values:57  values:85848  values:0  values:1  values:955506  values:213312  values:0  values:2  values:270652  values:22588  values:4  values:1457325  values:64566  values:4  values:20467  values:1  values:4  values:0  values:141992  values:32  values:100788  values:420  values:1  values:1  values:81663  values:32  values:59498  values:32  values:20142  values:32  values:24588  values:32  values:20744  values:32  values:25933  values:32  values:24623  values:32  values:43053543  values:10  values:53384111  values:14333  values:10  values:43574283  values:26308  values:10  values:16000  values:100  values:16000  values:100  values:962335  values:18  values:2780678  values:6  values:442008  values:1  values:52538055  values:3756  values:18  values:267929  values:18  values:76433006  values:8868  values:18  values:52948122  values:18  values:1995836  values:36  values:3227919  values:12  values:901022  values:1  values:166917843  values:4307  values:36  values:284546  values:36  values:158221314  values:26549  values:36  values:74698472  values:36  values:333849714  values:1  values:254006273  values:72  values:2174038  values:72  values:2261318  values:64571  values:4  values:207616  values:8310  values:4  values:1293828  values:28716  values:63  values:0  values:1  values:1006041  values:43623  values:251  values:0  values:1}}  prices:{steps:{numerator:721  denominator:10000000}  memory:{numerator:577  denominator:10000}}  max_execution_units_per_transaction:{steps:10000000000  memory:14000000}  max_execution_units_per_block:{steps:20000000000  memory:62000000}}

@verbotenj verbotenj requested a review from a team as a code owner November 7, 2024 13:38
@verbotenj verbotenj force-pushed the feat/readParams branch 2 times, most recently from e65337d to 5607a8d Compare November 7, 2024 13:43
agaffney
agaffney requested changes Nov 7, 2024
View reviewed changes
internal/utxorpc/query.go Outdated
@@ -63,23 +71,59 @@ func (s *queryServiceServer) ReadParams(
return nil, err
}

log.Printf("Get protoParams type: %T", protoParams)
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's remove this.

internal/utxorpc/query.go Outdated
return nil, fmt.Errorf("Utxorpc method not supported for current protocol parameters")
}

log.Printf("Converted protocol parameters to Cardano PParams: %+v", utxoParams)
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove this

@verbotenj verbotenj merged commit 518fd83 into main Nov 7, 2024
7 checks passed
@verbotenj verbotenj deleted the feat/readParams branch November 7, 2024 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@agaffney agaffney agaffney approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@verbotenj @agaffney