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

BitCircuits for pair of circuits #9

Open
3 of 12 tasks
khosravipasha opened this issue Nov 18, 2020 · 0 comments
Open
3 of 12 tasks

BitCircuits for pair of circuits #9

khosravipasha opened this issue Nov 18, 2020 · 0 comments
Assignees

Comments

@khosravipasha
Copy link
Contributor

Code used for demo benchmarking:

https://gist.github.com/khosravipasha/2e56013040be579993fbb65ed56f853d

Remaining tasks, check Tractables/ProbabilisticCircuits.jl#33 for more details.

  • Sanity tests for BitCircuitPair shows some failures (but getting correct results for expectations, need to investigate more)

  • Tests for helper functions

  • Leaf cleanup (see comment 1)

  • Separate out pr-constrain (fvalues) and expectation (fgvalues) part maybe? If it does not slow down too much. In any case, having pr-constraint by itself is useful to have.

  • Easy and Efficient Support for batching (by reusing memory etc)

    • See commnet 5, seems much faster
  • Refactoring

    • make it easy to use other kernels than expecatations
  • Moments

  • Pruning some pairs, some pairs might always be zeros, if we can gurantee that maybe we can remove some. Might help in reducing size.

  • Specialized Cases: For example if both cases circuits are identical.

  • Generalized Product

@khosravipasha khosravipasha self-assigned this Nov 18, 2020
@khosravipasha khosravipasha reopened this Nov 23, 2020
@khosravipasha khosravipasha transferred this issue from Tractables/ProbabilisticCircuits.jl Oct 17, 2021
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

No branches or pull requests

1 participant