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

Update the gecode interface for gecode 5.x #129

Merged
merged 1 commit into from
Mar 3, 2020
Merged

Update the gecode interface for gecode 5.x #129

merged 1 commit into from
Mar 3, 2020

Conversation

jamesjer
Copy link
Contributor

This is a one-way trip, unfortunately. You gain the ability to use gecode 5.x by losing the ability to use gecode 4.x. The preprocessor can deal with the code, but I don't know what to do about gecode.ampl. This resolves #109.

@fdabrandao
Copy link
Member

Which source did you use at mp/thirdparty/gecode to in order compile? We tried with 5.0 and 5.1 and we got several errors with both.

@jamesjer
Copy link
Contributor Author

Sorry, I overlooked the github email! I'm using 5.1.0, on an x86_64 Fedora 31 machine. What errors did you get with 5.1.0?

@mapgccv
Copy link
Contributor

mapgccv commented Mar 3, 2020

Thank you @jamesjer: once i update the tests, this seems to work for the current version (6.2) too.
I think that we'd simply note that only gecode 5+ is supported with the current mp repo; in the unlikely case that people want to use older versions, they'll simply have to check-out an older commit of this repo.

@mapgccv mapgccv merged commit 2a3de0b into ampl:master Mar 3, 2020
@jamesjer jamesjer deleted the gecode5 branch January 15, 2021 22:20
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.

Compatibility with gecode-5.0.0
3 participants