Skip to content

Conversation

@vplesko
Copy link
Contributor

@vplesko vplesko commented Oct 7, 2023

No description provided.

@mxgmn mxgmn merged commit bdf8a22 into mxgmn:master Oct 7, 2023
@mxgmn
Copy link
Owner

mxgmn commented Oct 7, 2023

Nice! Did you use AC4-style propagation?

@vplesko
Copy link
Contributor Author

vplesko commented Oct 8, 2023

Thanks for merging, that was much faster than I expected!

I used AC3 because I wasn't aware of the underlying theory behind constraint propagation. I'll be trying out AC4 now that I'm aware of it.

@mxgmn
Copy link
Owner

mxgmn commented Oct 8, 2023

I just noticed that you put a lot of effort into optimization. AC4 speeds it up significantly, especially for samples with many patterns.

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