-
Notifications
You must be signed in to change notification settings - Fork 15
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
query protocol-parameters does not show params introduced in Conway #729
Comments
This should probably be done like this, by adding the selected lines: In this manner, we will:
which will make people involved in this discussion happy: #569 (comment) |
Do you think this will make it into 8.23.0.0 release? |
@gitmachtl> No it won't make it to 8.23.0.0, but we can probably prioritize it for the next one. Would that be useful to you? |
@smelc hi, yes please. because if every change we have to adjust all the external tooling. and at some point it would be good to come to a solution that will stay for a while. but have i read this correct, the outcome protocol parameters json will simply be like a merged output of the existing one and the extra parameters related to governance? like: {
"collateralPercentage": 150,
"costModels": {
"PlutusV1": [
205665,
812,
1,
...
],
"PlutusV2": [
205665,
812,
1,
...
],
"PlutusV3": [
205665,
812,
1,
1,
...
]
},
"decentralization": null,
"executionUnitPrices": {
"priceMemory": 0.0577,
"priceSteps": 7.21e-05
},
"extraPraosEntropy": null,
"maxBlockBodySize": 90112,
"maxBlockExecutionUnits": {
"memory": 62000000,
"steps": 20000000000
},
"maxBlockHeaderSize": 1100,
"maxCollateralInputs": 3,
"maxTxExecutionUnits": {
"memory": 14000000,
"steps": 10000000000
},
"maxTxSize": 16384,
"maxValueSize": 5000,
"minPoolCost": 340000000,
"minUTxOValue": null,
"monetaryExpansion": 0.003,
"poolPledgeInfluence": 0.3,
"poolRetireMaxEpoch": 18,
"protocolVersion": {
"major": 9,
"minor": 0
},
"stakeAddressDeposit": 2000000,
"stakePoolDeposit": 500000000,
"stakePoolTargetNum": 500,
"treasuryCut": 0.2,
"txFeeFixed": 155381,
"txFeePerByte": 44,
"utxoCostPerByte": 4310,
"committeeMaxTermLength": 73,
"committeeMinSize": 7,
"dRepActivity": 20,
"dRepDeposit": 500000000,
"dRepVotingThresholds": {
"committeeNoConfidence": 0.6,
"committeeNormal": 0.67,
"hardForkInitiation": 0.6,
"motionNoConfidence": 0.67,
"ppEconomicGroup": 0.67,
"ppGovGroup": 0.75,
"ppNetworkGroup": 0.67,
"ppTechnicalGroup": 0.67,
"treasuryWithdrawal": 0.67,
"updateToConstitution": 0.75
},
"govActionDeposit": 50000000000,
"govActionLifetime": 6,
"minFeeRefScriptCostPerByte": 44,
"poolVotingThresholds": {
"committeeNoConfidence": 0.51,
"committeeNormal": 0.6,
"hardForkInitiation": 0.51,
"motionNoConfidence": 0.6,
"ppSecurityGroup": 0.6
}
} |
Yes this is the plan 👍 but to be confirmed when I deep dive on it 😆
Noted, will do this one soon 👍 |
@gitmachtl> current state vs new state looking like this in my PoC ( I think this matches your expectations, right? |
yea, nice. 👍 |
@gitmachtl> final version trims some fields that were Is that fine? |
|
@gitmachtl> to be explicit, the new version removes some |
yea, all good. a jq query would also report back a |
@smelc will the governance parameters still be there via the query gov-state command? or will that section in the output be removed? |
@gitmachtl> I only change |
@smelc please notice #758 (comment) |
@smelc , posted you some test lines #758 (comment) |
Description
We are missing the parameters introduced in conway in the output of
query protocol-parameters
Desired behavior
query protocol parameters
output should be similar toSteps to Reproduce
cardano-cli conway query protocol-parameters
Additional Context
Possible Solution
We need to return all parameters.
The text was updated successfully, but these errors were encountered: