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

Parse solver output of external solvers #94

Merged
merged 2 commits into from
Apr 29, 2024
Merged

Parse solver output of external solvers #94

merged 2 commits into from
Apr 29, 2024

Conversation

chrjabs
Copy link
Owner

@chrjabs chrjabs commented Apr 29, 2024

Description of the Contribution

Cleaned version of #86, mainly implemented by @atimaly.

PR Checklist

  • I read and agree to CONTRIBUTING.md
  • I have formatted my code with rustfmt / cargo fmt --all
  • Commits are named following conventional commits
  • I have added documentation for new features
  • The test suite still passes on this PR
  • I have added tests for new features / tests that would have caught the bug this PR fixes (please explain if not)

atimaly and others added 2 commits April 29, 2024 15:13
allow parsing of output of externally run SAT solvers

closes #86

SAT solver output parser

SAT solver output parser Error handling

Adding constructor and reorganizing code

Adding tests and small correction

Adding tests for empty solution and error

Adding tests for types and correction for parsing

Adding InvalidVline error and cleaning up logic

docs: Add simple documentation

File reading tests and other additions
@chrjabs chrjabs merged commit 81c27d4 into main Apr 29, 2024
44 checks passed
@chrjabs chrjabs deleted the feat/solver-output branch April 29, 2024 12:25
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