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

Using -t option with Chuffed kills the solver before statistics are printed #692

Closed
raphaelboudreault opened this issue May 9, 2023 · 2 comments
Labels
bug resolved Issue is resolved and the feature or fix will be part of next release

Comments

@raphaelboudreault
Copy link

raphaelboudreault commented May 9, 2023

Relates to chuffed/chuffed#85 and #523.

While I thought that MR chuffed/chuffed#87 and chuffed/chuffed#88 would solve this issue, it still happens on Windows 10 using MiniZinc 2.7.2. The flag -t is now passed correctly to Chuffed, but it seems like MiniZinc still kills Chuffed before it has the time to print anything. This can be seen using the flag -s, where Chuffed doesn't print any of its statistics when the timeout kicks in (when I'm pretty sure back in the days, it was doing so). Note that this behavior of missing output also happens when using Ctrl+C (only ***Interrupted*** is printed).

@raphaelboudreault raphaelboudreault changed the title Using -t option using Chuffed kills the solver before statistics are printed Using -t option with Chuffed kills the solver before statistics are printed May 9, 2023
@Dekker1
Copy link
Member

Dekker1 commented May 10, 2023

This seems like an issue in Chuffed, not in MiniZinc. Could you please recreate the issue there.

I just wanted to note that I cannot recreate the issue on macOS, but hopefully someone with Windows would be able to recreate it and help solve it.

@Dekker1 Dekker1 closed this as completed May 10, 2023
@raphaelboudreault
Copy link
Author

raphaelboudreault commented May 10, 2023

@Dekker1 Note that the behavior is correct when using fzn-chuffed alone (with option --time-out). Opened chuffed/chuffed#135, as requested.

@Dekker1 Dekker1 reopened this May 10, 2023
@Dekker1 Dekker1 added bug resolved Issue is resolved and the feature or fix will be part of next release labels May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug resolved Issue is resolved and the feature or fix will be part of next release
Projects
None yet
Development

No branches or pull requests

2 participants