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

Introduce type for domain values #76

Open
ChrisJefferson opened this issue Nov 16, 2023 · 1 comment
Open

Introduce type for domain values #76

ChrisJefferson opened this issue Nov 16, 2023 · 1 comment
Assignees
Labels
area::conjure-oxide Related to conjure_oxide. good first issue Good for newcomers

Comments

@ChrisJefferson
Copy link
Contributor

This doesn't need doing now, but might be worth doing if easy, to reduce future churn.

At the moment most of conjure-oxide is using i32 for integers. Minion supports 64 bit domains (if compiled with it), and z3 (picking a random example) supports any sized integer.

Might be worth adding a type decleration to define domain values, which could then be tweaked later, to carefully handle the various different limits of different solvers.

@ozgurakgun
Copy link
Contributor

Good idea, but let's save this for later. Could be a nice low hanging fruit for someone :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area::conjure-oxide Related to conjure_oxide. good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

4 participants