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

Added comments and explainer output. #19

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

dtonhofer
Copy link
Contributor

Added lots of comments to make this problem clearer.
Also added code to generate "explainer output" when a solution has been found.
Note that there is a typo which remains: "leave" instead of "leaf" but I left this as-is.

@dtonhofer
Copy link
Contributor Author

I can't believe it's been 3 years already, my brain has lost any knowledge about minizinc like a leaky sieve. Guess I will have to take the course again. Such is life in the zone.

But apart from that, I don't think this change will ever be taken up, should I drop my forked "specialization-examples" repository?

@Dekker1
Copy link
Member

Dekker1 commented Jun 20, 2024

I'm sorry that it has been so long and nothing has been done with your suggested improvements. It certainly is something that I would like to have a look at, but it is hard to find time to do so. I will see whether I can do so within the next month or so.

@dtonhofer
Copy link
Contributor Author

Thanks Jip. I know that feel.

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