-
Notifications
You must be signed in to change notification settings - Fork 155
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
Fix costmodel creation in Alonzo Genesis #3101
Conversation
83038f9
to
295e71b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great to me, thank you @teodanciu !
@Jimbo4350 - you should be aware that the CLI is going to need to use legacyParseCostModels
for parsing the original alonozo genesis file.
3a25ab9
to
670f5b5
Compare
instead of the plutus `ParamName` enum, since the latter returns slightly different names for 6 parameters: "blake2b_256-cpu-arguments-intercept", "blake2b_256-cpu-arguments-slope", "blake2b_256-memory-arguments", "verifyEd25519Signature-cpu-arguments-intercept", "verifyEd25519Signature-cpu-arguments-slope", "verifyEd25519Signature-memory-arguments" Instead of the names expected by the alonzo genesis file: "blake2b-cpu-arguments-intercept", "blake2b-cpu-arguments-slope", "blake2b-memory-arguments", "verifySignature-cpu-arguments-intercept", "verifySignature-cpu-arguments-slope", "verifySignature-memory-arguments" Also, replace the `FromJSON` instance with a function, to make it clear that it's a legacy implementation. respectively.
and the sorting of cost model parameters
which is needed for the future, non-legacy parsing of genesis and also for current testing.
670f5b5
to
a3c3451
Compare
Hello @Jimbo4350 - would you mind taking a look at this before we're merging it? Just to make sure it will work with your plans for testing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks 👍
No description provided.