-
Notifications
You must be signed in to change notification settings - Fork 483
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
Add verified compilation certification component #6413
Merged
Merged
Changes from 48 commits
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
dfde178
WIP
kwxm afa23a1
Merge branch 'master' into kwxm/metatheory/fix-test2
kwxm 21607ed
Fix broken metatheory tests using textual comparison
kwxm f18ebe7
Refactor detailed tests
kwxm 32c5a26
Refactor detailed tests
kwxm 9f17fba
Re-enable plutus-metatheory:test2 in CI
kwxm 4d32e29
Add raw certification component
ana-pantilie 540b95b
Merge branch 'master' into kwxm/metatheory/fix-test2
kwxm 225ac43
Fixes in project.nix
kwxm aa67c66
Remove unused files
kwxm 16d004c
Remove the preCheck hack from plutus-metatheory.components.tests.test…
zeme-wana 68652b3
Extra comments
kwxm 5411479
Merge branch 'kwxm/metatheory/fix-test2' of github.com:IntersectMBO/p…
kwxm b3d582b
WIP
kwxm a2dfbcd
Fix broken metatheory tests using textual comparison
kwxm 602d274
Refactor detailed tests
kwxm 48543fd
Refactor detailed tests
kwxm 2f43b7b
Re-enable plutus-metatheory:test2 in CI
kwxm 526e3eb
Fixes in project.nix
kwxm 5d177dc
Remove unused files
kwxm 8fe533f
Extra comments
kwxm 948bc91
Remove the preCheck hack from plutus-metatheory.components.tests.test…
zeme-wana fda7740
Rebase and remove dead code
zeme-wana 3568d60
WIP: fix compilation errors
ana-pantilie 89f9b08
Remove newline in error message
kwxm 35ee520
Merge branch 'kwxm/metatheory/fix-test2' of github.com:IntersectMBO/p…
kwxm bcb1607
Fix compilation errors
ana-pantilie a2445ad
Small clean-up
ana-pantilie e5ddb90
Address some PR comments
kwxm 3d9c46a
WIP
kwxm ee0e691
WIP
kwxm f389181
Add Output option to plc to make it consistent with uplc
kwxm 36f8947
Tidy up output of evaluation/typecheching results
kwxm 8c51c63
Tidy up output of evaluation/typecheching results
kwxm f9d5e2f
Name change
kwxm 7ff26ec
WIP: clean-up
ana-pantilie f52e423
Merge remote-tracking branch 'origin/kwxm/metatheory/fix-test2' into …
ana-pantilie 7778a19
Added plutus-executables package; moved uplc/pc/pir and metatheory tests
ana-pantilie 33b6482
Add certifier to uplc optimiser
ana-pantilie 1873aa8
Add certifier to uplc optimise
ana-pantilie 99bd8ba
Merge remote-tracking branch 'origin/master' into ana/add-certificati…
ana-pantilie 8d8d0a2
Fix
ana-pantilie 5733c83
Clean-up
ana-pantilie f451581
Clean-up cabal file
ana-pantilie 6c70e1c
Fix metatheory tests
ana-pantilie c24e54e
Fix isTransformation?
ana-pantilie d06b49b
Certifier properly handles results
ana-pantilie 9e42982
Add documentation
ana-pantilie 9c1b3e5
Merge remote-tracking branch 'origin/master' into ana/add-certificati…
ana-pantilie 74a2b0e
Fix dependency ranges
ana-pantilie 45136b6
Fix
ana-pantilie eac3c6b
Address review comment
ana-pantilie a04d246
Fix nix stuff
ana-pantilie 110e694
Add changelog
ana-pantilie 4823cff
Fix CI
ana-pantilie 14a861c
Maybe maybe
ana-pantilie 8ab0f74
Merge remote-tracking branch 'origin/master' into ana/add-certificati…
ana-pantilie 393695f
Fix master merge
ana-pantilie a2f5900
Fix
ana-pantilie File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
# Revision history for plutus-executables | ||
|
||
## 0.1.0.0 -- YYYY-mm-dd | ||
|
||
* First version. Released on an unsuspecting world. |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Why do you need this check here?
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.
If I recall correctly, it's to convert to the Agda representation of terms, the Haskell -> Agda intermediate module uses that:
plutus/plutus-metatheory/src/Untyped.hs
Line 36 in 4066409
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.
I don't get it. If it was for that function, you'd also need to check equality of
uni
andDefaultUni
. What breaks if you remove theeqT
check?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.
Good catch! I accidentally didn't use the
fun
type param in the SimplifierTrace type. Fixed now, thanks.