-
Notifications
You must be signed in to change notification settings - Fork 21
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
Cost models as list of integers #60
Comments
Worked on by #61, PR is not yet ready. |
It seems this is impossible to achieve when working with Aeson, see haskell/aeson#368. |
I have reached out to BF to either provide a new endpoint to give order of cost model parameter names (per plutus language) or to give cost models as a list of integers instead. |
Interesting! We decided to add |
Currently it is
newtype CostModels = CostModels { unCostModels :: Map ScriptType (Map Text Integer) }
, I propose it to benewtype CostModels = CostModels { unCostModels :: Map ScriptType [(Text, Integer)] }
since cost models are no longer ordered lexicographically. Attached is the cost models of PlutusV3 as returned by Blockfrost, and we notice that very last entry here starts with letterb
. Current type definition makes one lose the ordering as original returned by Blockfrost.Worth noting that cardano-ledger team in general recommends to keep cost models as list of integers.
The text was updated successfully, but these errors were encountered: