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

Quine-McCluskey Minimization Option #4

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

robertkleffner
Copy link
Contributor

No description provided.

@magnus-madsen
Copy link
Member

This is still marked a draft, but is it ready for review and eventual merge, or do you want to make substantial changes to it yet?

@robertkleffner
Copy link
Contributor Author

This is still marked a draft, but is it ready for review and eventual merge, or do you want to make substantial changes to it yet?

I will improve the documentation and make some parts of the code easier to read. It appears that the primeImplicants function is the big performance bottleneck right now, especially for equations with lots of min-terms, so if I can get a few optimizations in there I'll try that as well.

I think it will be ready for review by end-of-week.

@magnus-madsen
Copy link
Member

magnus-madsen commented May 22, 2022

@jacopol We should take a look at this when we find the time. Apparently the results look quite promising.

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