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

Current issues #4

Open
arbrad opened this issue Jan 4, 2015 · 0 comments
Open

Current issues #4

arbrad opened this issue Jan 4, 2015 · 0 comments

Comments

@arbrad
Copy link
Owner

arbrad commented Jan 4, 2015

  1. The comparator for stable_sort is passed by value, not by reference, which is a performance issue as a HeuristicLitOrder object holds substantial data. [31 May 2015: Fixed by mgudemann.]
  2. micAttempts is currently set to 3. It should be set to something much higher to allow more aggressive (but more expensive) inductive generalization.
  3. One-SAT-context-per-frame is inefficient, as it turns out. There should be one context for all frames.
  4. The logic cone should be loaded on-demand, not all at once.

Issues (1) and (2) are quick to fix, although I'm not currently maintaining the code.

Issues (3) and (4) are somewhat fundamental to the implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant