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

Add "benchmark" test #228

Merged
merged 13 commits into from
Feb 23, 2024
Merged

Add "benchmark" test #228

merged 13 commits into from
Feb 23, 2024

Conversation

khieta
Copy link
Contributor

@khieta khieta commented Feb 21, 2024

Issue #, if available:

Description of changes:

  • Update cedar submodule to the latest main branch
  • Restructuring/renaming in src/cedar_test_impl.rs, and corresponding updates to fuzz/src/lib.rs and src/lean_impl.rs.
  • Add an implementation for the CedarTestImplementation trait that uses our Rust library functions.
  • Add a new benchmark.rs test that runs the Lean & Rust implementation on the corpus tests.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@khieta khieta changed the title Khieta/add timing test Add "benchmark" test Feb 22, 2024
@khieta
Copy link
Contributor Author

khieta commented Feb 22, 2024

Not sure why CI is failing. I'm working on some other things right now, but I should be able to come back to this PR next week. Happy for any preliminary reviews in the meantime.

@khieta
Copy link
Contributor Author

khieta commented Feb 22, 2024

CI is fixed thanks to @cdisselkoen.

@khieta khieta merged commit e26a70c into main Feb 23, 2024
2 checks passed
@khieta khieta deleted the khieta/add-timing-test branch February 23, 2024 13:58
This was referenced Feb 23, 2024
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

Successfully merging this pull request may close these issues.

3 participants