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 contracts for SmallSort #234

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ShoyuVanilla
Copy link

Towards #56

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

@ShoyuVanilla ShoyuVanilla force-pushed the challenge-8 branch 4 times, most recently from 3486135 to 910cdea Compare December 29, 2024 16:45
@ShoyuVanilla ShoyuVanilla force-pushed the challenge-8 branch 8 times, most recently from 09b91ee to c8291b1 Compare December 30, 2024 16:28
@tautschnig
Copy link
Member

@ShoyuVanilla Would you be able to revisit this PR?

@ShoyuVanilla
Copy link
Author

@tautschnig I've tried some different approaches - mostly on my local repo, so not included in this draft - but most of them ran out of memory or took several tens of minutes due to the problem's complexities and kani's missing features. I'm planning to make some PRs to kani or learn verifast to deal with this. Would it be better to close this PR for now?

@tautschnig
Copy link
Member

@tautschnig I've tried some different approaches - mostly on my local repo, so not included in this draft - but most of them ran out of memory or took several tens of minutes due to the problem's complexities and kani's missing features. I'm planning to make some PRs to kani or learn verifast to deal with this. Would it be better to close this PR for now?

Thank you for the immediate response! Entirely up to you how you deal with this PR, just wanted to make sure it hasn't been abandoned.

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.

2 participants