-
Notifications
You must be signed in to change notification settings - Fork 2
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
Essence Problem Models Not Supported for generating Discriminating Instances #5
Comments
It appears to be fully functional for the tiny.essence problem model, but for more complicated ones such as car-sequencing has another issue. |
hi @vincepick, thank you for the updates. I guess SR doesn't support ortools yet, but to double-check, could you please post the detailed conjure/SR/solver command that caused the issue (produced when running the wrapper.py script), and the error message? |
Try |
Yes I can, I will take another look at it myself too I’ve just been trying to start documenting the issues more consistently using GitHub actions. This is an odd problem too because I know that ortools works for discriminating instances for the MiniZinc problem models, so I think that SR does support it, it may just be a part of wrapper causing an issue. I also made some changes to wrapper which fixed another bug but could have caused this one.
Vincent Pickering
…________________________________
From: Nguyen Dang ***@***.***>
Sent: Thursday, October 24, 2024 10:08 AM
To: stacs-cp/AutoIG ***@***.***>
Cc: Vincent Pickering ***@***.***>; Mention ***@***.***>
Subject: {Disarmed} Re: [stacs-cp/AutoIG] Essenc Problem Models Not Supported for generating Discriminating Instances (Issue #5)
hi @vincepick<https://github.com/vincepick>, thank you for the updates. I guess SR doesn't support ortools yet, but to double-check, could you please post the detailed conjure/SR/solver command that caused the issue (produced when running the wrapper.py script), and the error message?
—
Reply to this email directly, view it on GitHub<#5 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/A5XUL22IEXXEAX6OWCLPGYDZ5C2H3AVCNFSM6AAAAABQPM4XQOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZUG4YTMMZQGE>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
[ { ***@***.***": "http://schema.org", ***@***.***": "EmailMessage", "potentialAction": { ***@***.***": "ViewAction", "target": "#5 (comment)", "url": "#5 (comment)", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { ***@***.***": "Organization", "name": "GitHub", "url": "https://github.com" } } ]
|
Hi Dr. Dang and Dr. Oz, Thank you both for your advice, I tried running with my current approach again and think we should discuss this issue further in our meeting tomorrow @ndangtt. I'm afraid I am not positive which command you are referring to @ndangtt. I believe it was the following one, and this is the one I was examining, but please do let me know if you meant another: Its output:
I found the dictionary where the solver infos are defined in essence_pipeline_utils.py, and ortools indeed isn't listed. I can try adding it and then going from there, but am not sure that is the best approach. I believe we should discuss options and Dr. Oz's recommendation in our meeting tomorrow then go from there, I have quite a few questions and also want to confirm that what I already changed doesn't seem to introduce any more bugs before working off of it.
Thank you, |
Hi Dr. Dang, Thank you for meeting with us today. Here is the command and attached files we talked about. Please let me know if there is anything else I should add and I will get to it right away. I had to attach the files as a zip because GitHub doesn't recognize any of the file formats. In the meantime, I will continue to experiment by comparing the MZN implementation for generating discriminating instances with the Essence implementation. Thank you, Command: Command Output (With additional verbose flag):
|
thanks, @vincepick! @ozgurakgun: can we ask for your help on this please? We're using conjure's latest release and cannot make or-tools works via The files are attached in previous message by Vincent. When I switch to |
the release may be too old, you could try the latest docker file? https://github.com/conjure-cp/conjure/pkgs/container/conjure/285546501?tag=main specific versions can also be selected here: https://github.com/conjure-cp/conjure/pkgs/container/conjure/versions example command to run this via podman: https://github.com/conjure-cp/conjure-aas/blob/c0c94ad615b1540dde4169a32d1a5790cccddcd6/conjure-aas.js#L94 a major advantage is that you won't have to install any of the tools. alternatively you can build the latest version on main alternative 3: I can make a new release, I probably should! |
also, the binary was renamed to |
My container is built off of the most recent Conjure base image, so it should be fully up to date for Conjure unless it has had any updates in the last month or so. (It's using v2.5.1, 2024-09-25 to be specific) If there have been updates, I would be happy to rebuild my container and see if that makes any difference. Also, I have been having some odd new issues. Perhaps they are related to the renamed binary, thank you for pointing it out! Vincent |
if you are building on top of that docker image everything should work. what error message do you get? |
I tried remaking my current container earlier today but seemed to be running into issues with the path. I think they're probably setup errors on my end that will be quickly fixed, but I'll keep an eye out for if it may be related to the renamed binary when I start poking around at it again tomorrow. |
thanks very much @ozgurakgun! @vincepick: I think there are a couple of issues here:
|
@vincepick: another thing that I forgot to mention is the flag for time limit. We should use |
Sounds good, thank you for all the suggestions @ndangtt . I will try experimenting with all of them and update you tomorrow afternoon. Could you also please give us permissions to open a GitHub "project" if possible, so we can set some internal deadlines and tie them to the issues to document our progress more clearly? I saw you gave us "maintainer", but unfortunately it still doesn't let us open one. I am also happy to use another issue tracker if preferred! |
@vincepick: I'm not sure how to give you permission to create a project. If you find a guidelines on how to do it, please let me know and I'll try. Otherwise, if you can tell me what you want me to create, I can create one and add you and Tudor into it. |
After fixing the initial issue of key error for SRTimeLimit, it now says that solver ortools is not yet supported when trying to run it for discriminating instances.
The text was updated successfully, but these errors were encountered: