-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Move SFGS, CSG and Elliptical Slice samplers to pymc3-experiments #3353
Comments
So this would mean removing all the work done in #2544? |
@Akamoha Yes, alongside the SFGS. |
Could we move to a new repo maybe? (pymc-devs/experiment?) Along with Elliptical Slice Sampling. |
@junpenglao I like that. |
Awesome ;-) |
I can finally dump all my codes on Laplace approximation etc there :-||||| |
@Akamoha Do you want to take this one on? |
hey, @twiecki can I work on this one? |
Yes! thank you for volunteering.
…On Fri, Feb 1, 2019, 06:17 Harshit Bansal ***@***.*** wrote:
hey, @twiecki <https://github.com/twiecki> can I work on this one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3353 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AApJmLghZNDBYGWve3msOZGPvrIQ4KqRks5vJCJcgaJpZM4aQrlC>
.
|
ESS should not be experimental, and is useful for GPs. One of the demo notebooks uses it. |
I remember @ColCarroll was working on this https://github.com/ColCarroll/l2hmc_pymc3 |
@fonnesbeck In that case maybe we should have some more example of using ESS - eg how it is used with other Samplers |
@ferrine just added a license so it can be copied/moved with or without my permission (whoever wants to move it has my permission, but prominent acknowledgments to the authors should stay close to the code!) I can also move it... soon. |
Hi @twiecki, Can I work on this issue? |
@vbsinha Definitely. You can leave Elliptical Slice here. To the extend that whole files can be moved that's what I'd do. Ideally we can still retain the functionality in that repo, it should just require redirecting imports to pymc3 instead of relative imports. |
The NBs should be ported too. Basically the idea is that there is no mention of SGFS or CSG in pymc3, but if someone wants to try them, they can install pymc3-experimental. |
Would there be a way to index the doc on the website still? Maybe under a tab call Experimental? |
@junpenglao https://github.com/pymc-devs/pymc3/blob/master/docs/source/api.rst |
Closed by #3391. |
These three samplers are not well supported and do not appear to really be used. They are mostly experimental and should be in the main software. Thus, they should be moved to https://github.com/pymc-devs/pymc3-experimental alongside any tests and docs (tests and docs do not need to compile/function). If someone picks them up and demonstrates their usefulness they can certainly make their way back, but they have lied dormant for too long.
The text was updated successfully, but these errors were encountered: