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

New package: BeliefPropagation v0.1.0 #116763

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JuliaRegistrator
Copy link
Contributor

@JuliaRegistrator JuliaRegistrator commented Oct 7, 2024

Copy link
Contributor

github-actions bot commented Oct 7, 2024

Hello, I am an automated registration bot. I help manage the registration process by checking your registration against a set of AutoMerge guidelines. If all these guidelines are met, this pull request will be merged automatically, completing your registration. It is strongly recommended to follow the guidelines, since otherwise the pull request needs to be manually reviewed and merged by a human.

1. New package registration

Please make sure that you have read the package naming guidelines.

2. AutoMerge Guidelines are all met! ✅

Your new package registration met all of the guidelines for auto-merging and is scheduled to be merged when the mandatory waiting period (3 days) has elapsed.

3. To pause or stop registration

If you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.

Tip: You can edit blocking comments to add [noblock] in order to unblock auto-merging.

@JuliaRegistrator JuliaRegistrator force-pushed the registrator-beliefpropagation-f47dce41-v0.1.0-fd34961e7e branch from 6149a7c to 4ad8c17 Compare October 7, 2024 14:54
JuliaRegistrator referenced this pull request in stecrotti/BeliefPropagation.jl Oct 7, 2024
UUID: f47dce41-c8ad-4b69-b8b4-f2a36c7ebdf8
Repo: https://github.com/stecrotti/BeliefPropagation.jl.git
Tree: 1bd4900dee137bed5941bf2c9de948988508d138

Registrator tree SHA: 17aec322677d9b81cdd6b9b9236b09a3f1374c6a
@JuliaRegistrator JuliaRegistrator force-pushed the registrator-beliefpropagation-f47dce41-v0.1.0-fd34961e7e branch from 4ad8c17 to 61af613 Compare October 7, 2024 15:55
JuliaRegistrator referenced this pull request in stecrotti/BeliefPropagation.jl Oct 7, 2024
@gdalle
Copy link
Contributor

gdalle commented Oct 7, 2024

[noblock]
Hi @stecrotti, congrats on the package registration!
Just wanted to check if you were aware of the existence of RxInfer.jl?

@stecrotti
Copy link

stecrotti commented Oct 8, 2024

[noblock]
Hi @gdalle! I think I'd seen it some time ago but wasn't on my mind when I wrote the "see also" section in the readme. Ours is meant to be lightweight and easy to use for quick implementations of BP starting from the graph and the factors, there is little code which i hope is transparent.
I've been reading the docs and the source code for RxInfer and ReactiveMP. They are of course much richer but also i find less accessible for a first time user. For example it's not clear to me how to do plain BP without being in an inference context.

@gdalle
Copy link
Contributor

gdalle commented Oct 8, 2024

[noblock]
Alright, thanks for your answer, just wanted to make sure that there was no useless duplication of efforts! If you put [noblock] in your comment you'll allow this PR to proceed.

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

Successfully merging this pull request may close these issues.

3 participants