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.

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

Check (store, macro) tests using the Lurk Alpha Evaluator have an equivalent #878

Closed
huitseeker opened this issue Nov 10, 2023 · 5 comments

Comments

@arthurpaulino
Copy link
Member

arthurpaulino commented Nov 10, 2023

So, the concept of opaque pointers didn't materialize explicitly in LEM.

An opaque pointer can be expressed by a Ptr::Atom whose hash (hopefully) corresponds to the hash of some other pointer (potentially with children). At any point, if we need an API to create opaque pointers, we implement it by calling Store::hash_ptr to get the corresponding ZPtr and then translate it back to a Ptr::Atom with the same tag and value.

About those macros, I feel like we won't need them anymore, given our current testing suite, which follows the same standard without the need for those macros.

@porcuquine
Copy link
Contributor

I may not follow completely, but its important for the Lurk-data manipulation API to make it easy to create opaque pointers. Are you suggesting this will be lost?

This isn't just about testing. Even though we created formal commitments to provide a 'better way', it was useful and correct that we could roll our own poor-man's hiding commitments using Cons values. This (and it's just an example) required the verifier to be able to construct claims involving opaque Cons pointers.

@arthurpaulino
Copy link
Member

arthurpaulino commented Nov 10, 2023

Not lost. What I meant is that a function to create opaque pointers could be added on demand (and it's almost an one-liner). But I added it to #879 regardless.

@huitseeker
Copy link
Contributor Author

AFAICT, #879 fixes the equivalence for the first three, and the last two are covered by evaluation tests.

@huitseeker
Copy link
Contributor Author

Closed in #879

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants